-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Feature: Interactive Selector Tools #5786
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Ready for review. |
I've still got more comments to write, including the |
I decided to do all the tools at once (and add the paint tool), because it will better inform what the base class should look like. I'll also add a UAT notebook. |
@pelson @tacaswell, I propose the following: |
Closing in favor of implemented the Painter widget only in #7411. I bit off more than I can chew with this one. |
See discussion in #5782.
Implements
RectangleTool
,EllipseTool
, andLineTool
to flesh out the base class requirements. I'll save Lasso, Polygon and Span for subsequent PRs.