Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Multiple selection in GUI #52

Closed
bgribble opened this issue Mar 14, 2013 · 1 comment
Closed

Multiple selection in GUI #52

bgribble opened this issue Mar 14, 2013 · 1 comment
Milestone
mfp 0.03

Comments

@bgribble
Copy link
Owner

Currently, PatchWindow.selected is expected to be a PatchElement or None. This needs to allow a list of PatchElement.

Other changes to support multiple selections:

  • Additional keybindings to add, subtract, or replace selections in the list
  • Rethink input modes for selected objects. Sometimes we want to perform the same
    operation on all selections, but some operations make no sense in the group context.
    How is input handled when multiple objects are selected?
  • Sometimes multiple selection is an expansion of single selection. Sometimes it's a
    narrowing of "select all". When is each interpretation appropriate?
  • Rubberband box mode for selections
bgribble added a commit that referenced this issue Mar 21, 2013
@bgribble
GitHub #52: First checkpoint on multiselect. Not working yet.
581ff8d
bgribble added a commit that referenced this issue Mar 21, 2013
@bgribble
GitHub #52: Work on connection-mode, autoplace-mode, and select/unselect
fe83b48
bgribble added a commit that referenced this issue Mar 21, 2013
@bgribble
GitHub #52: Multiple select mostly complete, still a couple of issues
15a7bd1
bgribble added a commit that referenced this issue Mar 22, 2013
@bgribble
GitHub #52: Pass through initial click-to-select to underlying edit mode
29948d0
@bgribble
Copy link
Owner Author

Everything seems to be working. PatchControlMode is still a stub and doesn't have rubberband functionality yet, but PatchEdit mode works

bgribble added a commit that referenced this issue Mar 25, 2013
@bgribble
GitHub #52: Handle transient-message with multiple selected
a53be29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
mfp 0.03
Development

No branches or pull requests
1 participant
@bgribble