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

Refactor UI Layout #120

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

Conversation

imprisonedmind
Copy link

@imprisonedmind imprisonedmind commented Aug 12, 2024

General Overhaul of main UI Layout pattern:

  • Move away from using absolute positioning.
  • Start to move away from .css files.
  • Follow component based structure && rely only on tailwindcss.

Overhaul of Side Menu:

  • Side Menu open/closes on mouse enter state.
  • Simplification.
  • Mobile Responsive.
  • No use of ToolTip.

Update to Preferences:

  • Wrapper components to control sub list elements.
  • Chevron animates clockwise instead of anti clockwise.
    • Closed state is 0deg rightwards facing.
    • Open state is 90deg downwards facing.
  • Custom component for password inputs.
  • Custom component for color information.

Other:

  • Custom loading + spinner component.

// notes

  • SideMenu needs some extra TLC in terms of centering of icons when associated text is hidden. not 100% correct yet!

@imprisonedmind imprisonedmind changed the title Refactor UI layout Refactor UI Layout Aug 12, 2024
@megascatterbomb
Copy link
Contributor

Tooltips are supposed to appear when hovering over these icons, but they don't appear to work.

image

There's also still a scrollbar for the entire player history page, which partially occludes either the search bar or the bottommost result in the archive.

image

Other than that, it's looking good!

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