-
Notifications
You must be signed in to change notification settings - Fork 39
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
Move actions from modules to tray #1044
Move actions from modules to tray #1044
Conversation
One thing to discuss is the order of the tools, many users remember that the launcher was first from the top, now it is second... |
@iLLiCiTiT is there a way to keep @m-u-r-p-h-y thougts? here is the original order: and the new one: P.S.: I love the fact |
I can't keep it in previous order, I can move it under addons actions of above addons actions, both can be confusing. If it is under addons actions then Publisher is at different place. If it is above then e.g. ftrack would be at different place. |
do not understand the reasoning, but if not possible, then we can do nothing. I would also rather keep the same order from the previous one as @LiborBatek suggests and thanks for the |
also @MustafaJafar should be aware of the changes as it will affect the documentation . . . |
Note: The order of addon actions was defined by title of the addon, so changing Long story short, making this better means that we have to make something a little bit worse. |
Could we mark the menu entries with custom data like # psuedocode
def sort(action):
if action.order is not None:
return action.order
# sort by label
action.text I do think the Publisher is not so high priority over the Launcher that it should be at the top - and hence I feel the change is very confusing and we should look into ways to do set a preferred order. |
That would require some api for actions, which we don't have.
And should be launcher always at the top? Meaning that ftrack would move under it? (Can be achieved even now.) |
I personally would say so. But definitely go based off of the opinion of others here please. |
|
And |
yes, I dont mind being it at the very bottom for |
Changelog Description
Moved
Launcher
,Loader
andConsole
directly to tray actions and removedayon_core.modules
. Also addedPublish report viewer
action toAdmin
section.Additional info
Tools should work as before. Changed label of
Loader
toBrowser
, we've discussed it multiple times and if I remember correctly we came to conclusion that it should be Browser. AddedPubish report viewer
action which was requested multiple times.Because the actions are not defined by "addons" their order is not exactly the same.
Testing notes:
Resolves #858