-
Notifications
You must be signed in to change notification settings - Fork 374
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
Minor corrections in the menu #1631
Conversation
mkz212
commented
Nov 5, 2023
- Adding Logs to the menu.
- Change the name from Settings to Config Ui Settings
- Small change of order.
Do you have a screenshot of the differences? |
I will be very happy to do screenshoot. Just tell me where on FTP is the Config Ui plugin directory? It is intended to be something like this:
or
What do You think? Which is better? The longer I look at it, the more I have the impression that the second option is better. |
Yeah 2nd option |
@donavanbecker Please tell me what is the path to Config Ui folder? |
ok I found folder Anyway, I just update code to second option. |
My thoughts: do we need the That said, if we removed this, we could simply rename the 'Homebridge Settings' to 'Settings'. I asked chatgpt about ordering! The ordering of menu options in a drop-down menu can significantly impact user experience. Here's a suggested ordering for the menu options you provided based on their common usage and logical flow: User Accounts: Placing this option first allows users to manage their accounts, which is often a primary concern for users interacting with a system. Settings: Users typically access settings to customize their experience, so this option logically follows user accounts. Backup/Restore: Backup and restore options are usually related to data management. Placing them here allows users to manage their data before performing actions like restart or shutdown. Terminal: The terminal option is often used by advanced users or developers. Placing it after basic user-related options keeps the menu organized for both regular users and advanced users. Restart: Restarting the system is a common action. Placing it here ensures that important tasks like managing accounts, customizing settings, and managing data are completed before performing a system restart. Shutdown: Shutting down the system is a critical action and is typically placed at the bottom of the list to avoid accidental clicks. Logout: Logging out is usually the last option as it allows users to end their current session. Placing it at the bottom ensures that users consider other options before choosing to log out. This ordering provides a logical flow from user-specific actions to system-related actions, with critical actions placed at the end to prevent accidental selection. However, the specific ordering can depend on the context and the target audience of your application. Always consider conducting usability testing with your actual users to determine the most intuitive menu order for your particular interface. However I would suggest swapping the first two so that Settings comes first: ie
any thoughts? Having taken another look at my suggestion here, I also feel like shutdown server is too close to log out, could be accidentally clicked and a user may not know how to get it going again, or is away from home and cannot restart. Maybe we don't even need the shutdown option? Or we merge the two into a 'Restart/Shutdown' option and have the two buttons in a modal that appears, for an extra 'confirmation'. Maybe I am overthinking now :p |
I like the or we could just have a confirmation modal that pops up for both of them? |
|
I think the The confirmation modals for restart and shutdown already exist. I wonder why we need the extra logs item since there is an icon and link to the logs page at a higher level next to the 3 dots which opens the menu. Which I understand my thought here is inconsistent with the restart icon at the higher level, and also a restart menu item. Perhaps this whole section needs an overhaul |
|
👍
👍 Happy with the order, but Im still keen on keeping the text for the ui settings as
👍
Hmm Id rather keep it as it is, otherwise a user might simply it acts as a refresh button (as it looks like a browser refresh button) |
Change to UI Settings? I think it is even better. |
So mayby it is translation problem because in pl it is just "Ustawienia" ("Settings"). There is no "UI". |
Yeah it's probably just how it's been translated in the |
Head branch was pushed to by a user without write access
Ok now it is: UI + translated settings. UI is hardcoded. settings is translated. So in EN it will be UI Settings, in PL it will be UI Ustawienia. |