Skip to content
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

Tornjak UI Refactoring #551

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

EdgarAllan-Bro
Copy link
Contributor

I have refactored the main user interface for Tornjak. Please test for functionality.

I have refactored the main user interface for Tornjak. Please test for functionality.

Signed-off-by: Kai Chen <[email protected]>
@mamy-CS
Copy link
Collaborator

mamy-CS commented Nov 4, 2024

Thank you for this, I like the new look. Some comments below:

  • Top title with green background (Please remove to avoid redundancy and for a cleaner look)

  • When the left main Tornjak menu is collapsed, have the text of the respective menu displayed when user hovers over the icon (For example: Cluster Management)

  • On the previous UI (On the top right corner) - there used to be 3 icons (User Management, search, and notification icons please add them back

  • On Tornjak Dashboard page please make the side menu collapsable as it used to be

  • On Tornjak Dashboard - The title for the two pie charts is not there - add them back

  • There are some warnings on the console - mostly unused imports and variables please clear warnings

  • When you enable Health check - it messes up the UI please add Health check UI under server ID in the menu bar

  • When creating an entry - The notifications background is not showing fully

  • When scrolling down on the UI - Keep the left side menu fixed and only scroll the rest of the page [You can test this functionality by going to "Server Info" page and by expanding the verbose config section

  • And the UI needs to be thoroughly tested by enabling user management as well.

Fixes all the above. Let me know if there are any remaining issues that needs to be addressed.

Signed-off-by: Kai Chen <[email protected]>
Added text and resize on hover, health-check back, icons back

Signed-off-by: Kai Chen <[email protected]>
Allow more hover space for user logout, improve health check UI, and center title

Signed-off-by: Kai Chen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants