-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Final UI Changes #4080
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
Final UI Changes #4080
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR implements final UI changes to the CSS styling for the Espruino App Loader, representing a comprehensive redesign and reorganization of the stylesheet. The changes modernize the visual appearance with rounded corners, improved spacing, animations, and better responsive behavior. This work is a prerequisite for corresponding changes in EspruinoAppLoaderCore.
Key Changes:
- Complete CSS restructure with improved organization (navigation, buttons, tabs, app tiles sections)
- Enhanced visual design with rounded borders, shadows, animations (favorite button squish effect)
- Improved responsive layout with flexbox-based app tiles and better mobile support
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Removed border-radius from tab items and active tab.
|
Actually, before we merge, I wanted to try out a feature where app tiles that need to be updated show in a different color, just to differentiate better and see at a glance what needs to be changed. I'll work on that, and let you know the results |
|
This looks good to merge then! |
|
Thanks! I'll just wait until espruino/EspruinoAppLoaderCore#82 is sorted and then will put it all in together |
|
Merging now as espruino/EspruinoAppLoaderCore#82 is in |
This is the final, mergeable version of #4078
This must be merged before the corresponding
index.jschanges inEspruinoAppLoaderCore, #82