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

Change MenuText of fasteners according to grouping mode #363

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hasecilu
Copy link
Contributor

When the icons are not grouped in drop-down list the tooltip that appears is bloated/redundant, this changes expect to have cleaner tooltips.

image

Fix #355

@shaise
Copy link
Owner

shaise commented Apr 16, 2024

It looks like mode 2 misses the command name.

@hasecilu
Copy link
Contributor Author

I think that's the behavior of drop-down group, it was the same even before this change.

@shaise
Copy link
Owner

shaise commented Apr 16, 2024

Hi, Yes I completely agree with the changes to mode 0 and 1.
regarding mode 2, I don't think we need the help line because its a copy of the menu line. perhaps we need to just show the type + command name? i.e. ISO 4014 (FSISO4014)
What do you think?

@hasecilu
Copy link
Contributor Author

Do you know how to fix this? Could happen with other languages. When the tooltip is "" I think the MenuText is copied to ToolTip.
image

Something like this?
image

@shaise
Copy link
Owner

shaise commented Apr 16, 2024

  1. There is an issue with the tooltip in mode 2: when first time you point on the icon it shows title tooltip, but as soon as you select any fastener, the to tooltip changes. Dont know how to solve this.

  2. I want to show the command for script usage (same as other modes) so un this case it will be FSDIN653

@hasecilu
Copy link
Contributor Author

My main concern was the accent that was being not rendered correctly.

It seems in the previos image the second line was not the tooltip, is the "command", I remembered, when the tooltip is the same as the menutext it's hidden.
The last line is the command, it's in italic, and I suppose from FreeCAD is not expected to have accents.

image

This happens in the menu in Mode 2, somewhat bloated
image

VS mode 0, 1
image

When the icons are not grouped in drop-down list the tooltip that
appears is bloated/redundant, this changes expect to have cleaner
tooltips.

Fix shaise#355
@hasecilu hasecilu force-pushed the enhancement/change_menutext branch from 656c294 to cafc456 Compare April 16, 2024 17:38
@shaise
Copy link
Owner

shaise commented Apr 16, 2024

this is strange, I dont have this issue in mode 2:
image

@hasecilu
Copy link
Contributor Author

Try "Nut" group

@shaise
Copy link
Owner

shaise commented Apr 16, 2024

Yes, there are too many nuts.
We need to separate nuts to several groups, or find a new way to add fasteners.

@hasecilu
Copy link
Contributor Author

hasecilu commented Apr 16, 2024

Could be split on: "Hexagonal nuts", "Flange nuts", "Square nuts", "Cap nuts", "Wing castles", "Castle nuts" OR "Hexagonal nuts" and "Non Hexagonal nuts"

@czinehuba
Copy link

How about a different approach then separate commands in the toolbar. Fusion recently introduced a nice feature. The filtering and selecting it from three view looks really streamlined.

https://www.autodesk.com/products/fusion-360/blog/autodesk-fusion-fasteners-library/

@hasecilu
Copy link
Contributor Author

I think that major changes that impact on UI/UX should wait a time to see if FreeCAD improves the UI/UX and creates new guidelines for a better consistency.

I like the "stack" approach to attach washer and nut to a bolt, maybe in the future.

@shaise
Copy link
Owner

shaise commented Apr 26, 2024

I think that major changes that impact on UI/UX should wait a time to see if FreeCAD improves the UI/UX and creates new guidelines for a better consistency.

Yes, indeed I agree.

@hasecilu
Copy link
Contributor Author

What changes are still needed here?

@shaise
Copy link
Owner

shaise commented Apr 26, 2024

In my opinion, no matter how we arrange the toolbar, with more and more fasteners, we will always reach a limit where the usability will be bad. I think that ultimately there should be a single Add button that opens a task with a tree of fasteners.
But I will wait for the stabilization of freecad's ui/ux and perhaps get some ideas from the DWG team.

@hasecilu
Copy link
Contributor Author

And regarding the tooltips, does they need tweaks? The "bloated" menus on mode 2 is a secondary effect of the grouping, I think users will, most of the times, prefer to create their fasteners using the button menus rather than using the menus, so it's kind of "hidden" for them.

@czinehuba
Copy link

The tree view with just one button is a great idea.....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Add X Fastener" commands redundant
3 participants