Skip to content

Update UI design#212

Open
anncy0413 wants to merge 2 commits intoemanuelef:mainfrom
anncy0413:master
Open

Update UI design#212
anncy0413 wants to merge 2 commits intoemanuelef:mainfrom
anncy0413:master

Conversation

@anncy0413
Copy link

@anncy0413 anncy0413 commented Jun 2, 2025

  1. Implemented Dark/Light Mode Toggle
  2. Refactored Button Layout with Improved Spacing & Interactions
  3. Added Smooth Page Transition Animation on Info Page

#206

@emanuelef
Copy link
Owner

emanuelef commented Jun 3, 2025

Thank you for your work on this!
The changes look great at first glance.
I’ll take a closer look and provide a full review later today.
Appreciate your contribution!

@emanuelef emanuelef added the enhancement New feature or request label Jun 3, 2025
"papaparse": "^5.5.3",
"react": "^19.1.0",
"react-dom": "^19.1.0",
"react": "^18.3.1",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally it would be nice to use latest react version, I think the problem you might have seen is fusionchart not being compatible with latest React. What I'm now doing is to run npm i --force to keep using React 19.

@emanuelef
Copy link
Owner

I like the new header background that makes it different than the graph background.
The height seems to be more than before and ideally it would be nice to reduce the header instead.
There are probably too many options and is not super user-friendly.

The new buttons are great with the icons.
On that I'm not sure about the UX for the last 30 days one, would it be better to make it two-state so then it turns into All ?
Or remove it all together.

On large screen the right side buttons are aligned to the right and not sure how it is in terms of usability.

The icon for the theme maybe could be moved in the header ?

Screenshot 2025-06-03 at 17 11 34

Screenshot 2025-06-03 at 17 11 46

Thanks again for the PR, I think with your help the UI/UX can be improved a lot.
I'm also impressed that you managed to navigate in all the messy code that needs a massive refactoring !

@emanuelef
Copy link
Owner

Not sure both progress bars are needed, maybe we can simplify with just one or use a different indicator?
Open to any other UI/UX improvement ideas you have!

@emanuelef
Copy link
Owner

@anncy0413 there have been some recent updates to the UI and styling. I’d really appreciate hearing your thoughts when you have a chance.
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants