Skip to content

Conversation

@Ixitxachitl
Copy link

Standardize Emote Dimensions to 16x16 for Visual Consistency
This PR standardizes all emote dimensions in emotes.h to a uniform 16x16 pixel size, replacing the previous inconsistent mix of 30x30 and 25x25 pixel emotes. Additionally, new emotes have been added with the same standardized dimensions.

Changes Made
Standardized existing emotes: Updated all emote dimensions from 30x30 or 25x25 to 16x16 pixels for consistency
Added new emotes: Introduced heart_smile, Heart_eyes, and other new emotes, all at 16x16 size
Visual consistency: Uniform sizing provides a more polished and cohesive UI experience across all screens
Benefits
Consistent visual design: All emotes now display at the same size, creating a cleaner, more professional appearance
Better UI alignment: Standardized dimensions ensure proper spacing and alignment in message displays
Improved maintainability: Uniform sizing makes it easier to add new emotes in the future and ensures they integrate seamlessly

I have thoroughly tested these changes on the following devices:

✅ Station G2
✅ LilyGo T-Echo
✅ M5Stack Cardputer (Advanced variant)

Updated the dimensions of various emotes in emotes.h from 30x30 or 25x25 to 16x16 pixels for consistency and optimization. Added new emotes including heart_smile, Heart_eyes, and others, all with the same 16x16 size. This change improves memory usage and aligns with the design specifications for smaller emotes.
Copy link
Contributor

@Xaositek Xaositek left a comment

Choose a reason for hiding this comment

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

A lot of great icons here; definitely ready to be used.

Copy link
Contributor

@HarukiToreda HarukiToreda left a comment

Choose a reason for hiding this comment

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

very great refactoring of emotes, smaller take less space. Tested on T114 and Heltec V3

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.

4 participants