Add scroll to menu in system administration page#193
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
tarekio
left a comment
There was a problem hiding this comment.
Let's revert to normal scroll. Also, the menu seems to be squeezed, let's restore previous look, and use primary color for selection.
…467-Add-scroll-to-system-setting-menu
|
Reverted to regular scroll, and changed to primary color |
tarekio
left a comment
There was a problem hiding this comment.
This is ok for now, we can improve more in the future.
level09
left a comment
There was a problem hiding this comment.
I think this is a good simple fix for the issue. but I think the UX is still not ideal, not because of CSS but because of MAC defaults (hides scrollbars) so there is no way for the user to tell if he can scroll of not.
Maybe we can implement a simple UI enhancement here ? like show a little tiny UI detail when there is a scrollbar?
Jira Issue
Description
Issue
On the System Administration page, the menu becomes partially inaccessible on low-resolution screens. During usability testing, some users didn’t notice the notification at all. Additionally, other panels can overflow on smaller screens, making some content hard or impossible to access.
Solution
Checklist
API Changes (if applicable)
Additional Notes
[Any other relevant information]