-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
↔️ Make app order configurable #4917
Comments
I certainly agree! 👍 Each user should be able to reaarange its app order resp. the default app at login ! |
But only if allowed to! 😄 |
So this would be another cool modification of @juliushaertl to include in the core server. :) Some considerations:
|
Adding one more point to your considerations, @jancborchardt
|
@pixelipo in that case the next app should take its place, probably simplest solution. Maybe there should be some feedback, but that's a slightly different topic. |
Because of recent discussion in #5135 any handling of maximum shown apps could be take into consideration as well - fixed or even in a more dynamic way, maybe manageable for each user. For example by moving the "more apps"-toggle to the left/right to show less or more apps. Or may it be out-of-scope of this implementation? If so, we could at least keep it in mind for any upcoming versions. |
@patschi that specifically is out of scope in general cause it's a very edge-case customization option which will mainly increase the amount of stuff we need to test and can possibly break. This, apart from the nicer UX, is a big reason why we reduce options and make the defaults great. :) |
One question, how can a user reset a customized app ordering? |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I will now hide every non-contributing comment. What you can do instead:
|
Also updated the original post with specification and answer to all considerations raised. |
Please add the possibility to change default app without having to go to config.php. This should be in GUI. |
from Julius:
|
Just found out that apporder doesn't and won't ever work on nc25+. Features I severely miss are the option to adjust the default order of apps both globally and per user, and the ability to hide app icons while leaving the app enabled. For instance I use the keeweb app to allow users to manipulate keepass databases they have in their NC, but I don't want/need the icon for the app cluttering up my app list for all users. Please consider integrating the full functionality of this app into the standard NC interface as these are features that seem extremely useful, by allowing admins to provide interfaces that are less complex for their users when possible. |
The only place where moving something using is possible using keyboard, is moving browser tabs: Ctrl + Shift + Page Up/Page Down. If we implemented changing order using keyboard, we would be more accessible than most apps! Maybe Ctrl + ←/→? It normally skips over words in things like Or maybe some "Edit app menu" button? After pressing that button, pressing Enter wouldn't launch an app, but instead would select the app and pressing left and right arrow would move it. Pressing Esc or Enter would exit the icon moving mode. The editing app menu mode could also be used for hiding icons in a similar way, customizing toolbar is is implemented in Firefox. When in that mode, a box with hidden app icons would show up below the app bar. To hide an app icon, just drag it to that box or press the ↓ key. |
@jancborchardt I was wondering if there is any roadmap or update on this functionality. The Apporder features are crucial to me and this lack of updates is hindering my transition to nc25. |
This app seems to provide the desired functionality and much more: https://apps.nextcloud.com/apps/side_menu |
Not really. If you don't want a new menu on the side just to configure the normal menu, it's not a solution, sorry. |
How can you have both? Did I miss a setting? |
Hah, really! In the admin settings, not the personal settings! Thank you very much, you made my day! |
You're welcome. It's like using a sledgehammer to crack a nut, but it solves the problem. |
Another possible workaround would be the use of the app "Custom CSS" [data-id="music"] {
display: none !important;
}
#appmenu [data-id="activity"] {
order: 999;
} or with NC25 [data-app-id="music"] {
display: none !important;
}
#app-menu [data-app-id="activity"] {
order: 1;
} |
This is an interesting workaround, however CSS cannot solve the issue about keyboard navigation like AppOrder does. See explaination here : The order property and accessibility We need either to reorder the navigation bar on server side or to inject custom JavaScript and modify DOM on page load 😕 |
Discussed with @AndyScherzinger and @karlitschek
@nextcloud/designers this would be ready for picking up for a mockup. :) |
I can take this over :) |
🎉🎉🎉 |
Edit by @jancborchardt with spec:
What you can do to help:
Admins should be able to
If users are allowed to change the order by themself, this configuration should be handles via drag-and-drop in the app menu.
We could use https://github.com/juliushaertl/apporder/ as base ...
Follow up: juliushaertl/apporder#29
@juliushaertl @nextcloud/designers
The text was updated successfully, but these errors were encountered: