Skip to content

Conversation

@RKBoss6
Copy link
Contributor

@RKBoss6 RKBoss6 commented Dec 4, 2025

This is the final, mergeable version of #4078
This must be merged before the corresponding index.js changes in EspruinoAppLoaderCore, #82

Copilot AI review requested due to automatic review settings December 4, 2025 21:00
Copy link
Contributor

Copilot AI left a 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.

Copy link
Contributor

Copilot AI left a 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.

gfwilliams and others added 4 commits December 5, 2025 09:06
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Removed border-radius from tab items and active tab.
@RKBoss6
Copy link
Contributor Author

RKBoss6 commented Dec 5, 2025

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

@RKBoss6
Copy link
Contributor Author

RKBoss6 commented Dec 5, 2025

This looks good to merge then!

@gfwilliams
Copy link
Member

Thanks! I'll just wait until espruino/EspruinoAppLoaderCore#82 is sorted and then will put it all in together

@gfwilliams
Copy link
Member

Merging now as espruino/EspruinoAppLoaderCore#82 is in

@gfwilliams gfwilliams merged commit 92a10ce into espruino:master Dec 9, 2025
1 check passed
@RKBoss6 RKBoss6 deleted the finalCssChanges branch December 9, 2025 12:22
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