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

Improve UI/UX in large screens #277

Merged
merged 2 commits into from
Dec 21, 2024
Merged

Conversation

enrique-lozano
Copy link
Owner

@enrique-lozano enrique-lozano commented Dec 21, 2024

Pull request (PR) checklist

Please check if your pull request fulfills the following requirements:

  • I've read and understand the Code Contributions Guide.
  • I confirm that I've run the code locally and everything works as expected.

Screen recording of your changes (if applicable):

Video.mp4

What's changed?

Design improvements for all the larger screen devices (Windows devices, tablets...). The improvements are:

  • Removed the rounded border in the left side navigation bar on desktop screens
  • Applied a custom color in the left side navigation bar (the same color as the cards) and remove the gray border divider
  • In the stats page, removed the calendar selector from the bottom and placed in the top-right. In that way, larger devices will have more space to see the charts
  • Pages with top tabs (stats page, budgets page and budget details page) now align the tabs to the left in larger screens

Does this PR close any GitHub issues? (do not delete)

No

@enrique-lozano enrique-lozano added type: enhancement New feature or request UI / UX Design-related tasks labels Dec 21, 2024
@enrique-lozano enrique-lozano merged commit 4c0ec4f into develop Dec 21, 2024
@enrique-lozano enrique-lozano deleted the feat/improve-large-screens-ux branch December 21, 2024 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement New feature or request UI / UX Design-related tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant