-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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. |
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: juliusknorr/apporder#29
@juliushaertl @nextcloud/designers
The text was updated successfully, but these errors were encountered: