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.

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

Feature : Group Handling improvements #147

Open
sconklin opened this issue Nov 23, 2017 · 4 comments
Open

Feature : Group Handling improvements #147

sconklin opened this issue Nov 23, 2017 · 4 comments

Comments

@sconklin
Copy link
Collaborator

From @ronanletiec on October 10, 2017 9:46

The groups are used in Draw-Mode in order to display or hide elements.

As of v.0.6.0, we have the following functionalities:

  • Add selected elements to a new group
  • Make group visible / invisible
  • Rename Group
  • Delete Group

For a better handling of the groups, we need in addition the following functionalities:

  • Add selected elements to an existing group
  • Remove elements from existing groups
  • Show All Groups
  • Hide All Groups
  • Ability to lock a group from editing or deletion

To integrate the functions, we propose the following changes:

  • improve the dialog to add the elements to a group:

    • a select box with the value "new group" and the existing groups
    • an input to enter the name of the new group
    • if "new group" is selected, then the input is active, otherwise it's inactive
  • a new dialog (called when right clicking on a group and select the new entry "edit")

    • input to edit the name
    • check box for the visible flag
    • list with the elements contained, in alphabetical order. Each element in the list has a delete icon.
    • Cancel / Save Buttons
  • The Tooltip in Operations should be called "Add to group" instead of "Create new group".

  • enhanced Group-Overview

    • A new Icon Bar, that can replace / work along with the context menu, to call the following functions:
      • Show all groups
      • Hide all Groups
      • Add new Group
      • Delete Group
      • Edit Group

The Undo / Redo functionalities have to be taken into account.

What do you think? If it's ok with you, then I would implement it.

Future Additions (can be gathered and then we make a new issue out of them)

  • Color Option to set all elements in a group to the same chosen color
  • Line Style Option to set all elements in a group to the same chosen line style
  • for both suggestion we have to take into account that an element can be in several groups at the same time.

Edit : I update the description as the discussion goes to have an up to date overview, of what we will impelement.

Copied from original issue: valentina-project/vpo2#765

@sconklin
Copy link
Collaborator Author

From @DSCaskey on October 11, 2017 1:9

I suggest the following addition:

  • Ability to lock a group from editing or deletion
  • Color Option to set all elements in a group to the same chosen color
  • Line Style Option to set all elements in a group to the same chosen line style

I find holding the CTRL key then clicking elements one at a time then finally return to end - akward. It would make more sense to click an object and it shows up in a list. Click OK (or SAVE) to save the New or Edited group.

group

@sconklin
Copy link
Collaborator Author

From @ronanletiec on October 11, 2017 8:34

I find holding the CTRL key then clicking elements one at a time then finally return to end - akward. I was thinking it would make more sense that you simply click an element to add and it shows up in a list. Click OK (or SAVE) to save the New or Edited group.

I agree that it's a bit akward. But at the same time it's consistent with the other operations / functionalities. Mmmh where do you suggest the list would be shown? Temporarly the place of the Group overview?

If I remember well, the "Workpiece tool" used to have another dialog opened on top of the program while selecting the points and elements. It wasn't very handy.

@sconklin
Copy link
Collaborator Author

From @DSCaskey on October 12, 2017 4:10

I agree that it's a bit akward. But at the same time it's consistent with the other operations / functionalities.

The main thing I find akward... or maybe confusing is more correct - is having to hit < enter > AFTER elements are selected, ONLY then the dialog pops up to name the group. By having a dialog that requires OK or Save makes it obviously apparent what to do. Plus in this case typing < enter > would perform the same function as hitting OK.

Mmmh where do you suggest the list would be shown? Temporarly the place of the Group overview?

If I remember well, the "Workpiece tool" used to have another dialog opened on top of the
programm while selecting the points and elements. It wasn't very handy.

All the tools have a popup dialog... I don't see any difference between that and having a popup to add a new group name and list of elements... a list that can be edited before selecting OK. It could probably just as well reside in the group docker.

BTW... I haven't specifically looked at the group code, but have investigated the Tool Options dialogs & docker (there are some inconsistencies & bugs I want to fix), but the code is a mix of Qt UI in the dialogs with propertyexplorer code within the docker??? not sure how the Groups are handled within the docker.

@sconklin
Copy link
Collaborator Author

From @ronanletiec on October 12, 2017 7:43

Unless you read the ahem manual, you may not know how to add a group.

Everytime you use the tool, a help is displayed at the bottom of the software and tells you exactly what to do. The way to do it is maybe not very intuitive, but it's not like you have to open a manual to know what to do. But I must say the first time I used the software, I didn't see the tooltip at the bottom and thus were a bit helpless! So you're right, it should be more intuitive.

All the tools have a popup dialog... I don't see any difference between that and having a popup to add a new group name and list of elements...

I see a major difference : for the other tools, once the dialog is opened, you can't (and in my opinion shoudln't be able to) interact with the main software. In the moment, the dialog is the main focus. So if a dialog opened but at the same time you need to select points in the software, it's no intuitive. It was like this for the workpiece tool, i didn't like it, always had to move the dialog while selecting the points.
But I do agree that it would be much nicer to actually see a list of the selected elements (other than the selection squares on the piece in the draw mode) while editing and before clicking OK. A list that can also be edited before clicking OK.
That's why I prefer your suggestion instead of a dialog:

It could probably just as well reside in the group docker.

BTW... I haven't specifically looked at the group code, but have investigated the Tool Options dialogs & docker (there are some inconsistencies & bugs I want to fix), but the code is a mix of Qt UI in the dialogs with propertyexplorer code within the docker??? not sure how the Groups are handled within the docker.

I don't know unfortunatly, I haven't dealt with the docker yet.

@slspencer slspencer added this to Needs triage in New Features - Draw mode via automation Aug 10, 2020
@slspencer slspencer changed the title Improve Feature : Group Handling Feature : Group Handling improvements Jan 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
New Features - Draw mode
  
Needs triage
Development

No branches or pull requests

3 participants