Skip to content

Addon: Added button to open folder with Addons #55

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

Merged
merged 2 commits into from
Apr 28, 2025
Merged

Addon: Added button to open folder with Addons #55

merged 2 commits into from
Apr 28, 2025

Conversation

tarman3
Copy link
Contributor

@tarman3 tarman3 commented Apr 28, 2025

1_hor

@chennes
Copy link
Member

chennes commented Apr 28, 2025

It's a good idea, but we've clearly got too many buttons down at the bottom there. Are you interested in a slightly larger refactoring that would compress everything but "Check for updates" into a single drop-down "gear" menu?

@tarman3
Copy link
Contributor Author

tarman3 commented Apr 28, 2025

Excuse me, I didn't understand what exactly you offer
Combine several functions into one button with a drop down list ?

@chennes
Copy link
Member

chennes commented Apr 28, 2025

Yes: to make "Refresh local cache", "Python dependencies...", and "Open Addons Folder" all into a single button with a drop-down menu. This is itself a stopgap solution, the Addon Manager has grown to the point where the whole UI needs to be redesigned, but that's going to be a larger team effort (of course).

@tarman3
Copy link
Contributor Author

tarman3 commented Apr 28, 2025

Can you tell a place in Freecad, where this has already been implemented?
My developer capabilities are very low

@chennes
Copy link
Member

chennes commented Apr 28, 2025

The other places it's done in FreeCAD are in C++, so they probably won't help you as an example. What's needed is a QToolButton with its ToolButtonPopupMode set to MenuButtonPopup, and then a QMenu attached to it. That might be enough to get your AI tool of choice to get you started, but if not I can merge this PR as-is and then make the adjustment in another pass.

@tarman3
Copy link
Contributor Author

tarman3 commented Apr 28, 2025

I prefer to merge as-is
Later I will try to solve this task with popup button

Thank you

@chennes chennes merged commit d2e6c14 into FreeCAD:main Apr 28, 2025
5 checks passed
@chennes
Copy link
Member

chennes commented Apr 28, 2025

OK great. I've merged the change, but haven't pushed it back to the main FreeCAD repo yet -- I have a few other things I'm working on now, but should be able to do that by the end of the week. Thanks again for the PR.

@tarman3 tarman3 deleted the folder branch April 28, 2025 18:12
@tarman3
Copy link
Contributor Author

tarman3 commented Apr 29, 2025

Screenshot_20250429_092956_lossy

Something turns out ))

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.

2 participants