-
Notifications
You must be signed in to change notification settings - Fork 10
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
Improve tooltips for icons so they actually describe what they do #22
Comments
ok, that's a good one. i'll put it at the top of my todo list |
I'll be happy to help. Also If I may, would you consider also formatting the code block in a more readable way? i.e.
This way when it comes time to translate this workbench, it will be easier to do. |
Maybe you have time to do this? |
@edwardvmills softest of bumps |
@wandrewkeech thanks for all your work on #28 Silk/ControlGrid3Star66_3Sub.py Line 48 in affc35d
Any specific reason why ? |
i think that command/object was an early prototype, completely subsumed by ControlGridNStar66 (which covers the case where N=3) maybe it could be deleted, but i probably left it there in case i wanted to pick things out of it. |
I just pushed a new system for tooltip + popup window for additional info by function. system is in place, 8/25 functions complete. hoping to complete this phase in one more week? it is very challenging to say what the tools do :( as a lot of background info is required, as well as building up to the next tool in sequence. right now i'm just putting extra info in the simplest Qt message window. long term i want to incorporate images and have the window scrollable. I also want the window to be dockable so it can be placed split screen with FreeCAD. Seems like i should be using Markup, not strings.... this was my very first Qt, so i want to take it slow. It's a step toward having all relevant info included in the workbench. |
Right now, many of the icons are just placeholder names of the icon name, for example: https://github.com/edwardvmills/Silk/blob/master/ControlGrid66.py#L57
The text was updated successfully, but these errors were encountered: