Skip to content

Asset Packs: Warning for packs that contain icons and fonts for RAM usage#344

Open
956MB wants to merge 3 commits intoNext-Flip:devfrom
956MB:feat/asset-packs-ram-warning
Open

Asset Packs: Warning for packs that contain icons and fonts for RAM usage#344
956MB wants to merge 3 commits intoNext-Flip:devfrom
956MB:feat/asset-packs-ram-warning

Conversation

@956MB
Copy link
Copy Markdown
Member

@956MB 956MB commented Jan 5, 2025

What's new

Asset Packs: Warning for packs that contain icons and fonts for RAM usage

Checks asset packs if they contain Fonts and/or Icons and adds "Size Warning" list entry, as well as a warning screen describing the ram usage of font/icons in the selected pack, and an info list of all the sizes of the files in the pack.

The information in the info screen is just about being extra informative. No actions can be done on any of the files shown.

Screenshots

Packs without fonts/icons, all is normal

_0000_Default

'Size Warning' entry added to the list in MNTM > Interface > Graphics

_0001_WarningEntry

Warning screen (optional 'Back' button to just not look at the size info)

_0002_SizeWarning

Size info for all font/icon files

_0003_WarningInfo


For the reviewer

  • I've uploaded the firmware with this patch to a device and verified its functionality
  • I've confirmed the bug to be fixed / feature to be stable

- Checks asset packs if they contain Fonts/Icons and adds "Size Warning" list entry that opens new scenes.
- First scene is just a warning with text explaining that fonts and icons remain loaded and use up memory.
- Second scene uses the file list widget to display all the selected asset packs fonts and icons, as well as their sizes. This information is just about being extra informative. No actions can be done on any of the files shown.
956MB added a commit to 956MB/Momentum-Firmware that referenced this pull request Jan 5, 2025
@956MB
Copy link
Copy Markdown
Member Author

956MB commented Jan 8, 2025

Aligned the text to the center to improve the look and closer match other screens like "Enable Debug?".

Before After
Before After

@956MB 956MB changed the title Asset Packs: Warning for RAM usage Asset Packs: Warning for packs that contain icons and fonts for RAM usage Jan 13, 2025
@956MB 956MB self-assigned this Jan 13, 2025
956MB added a commit that referenced this pull request Jan 2, 2026
956MB added a commit that referenced this pull request Jan 2, 2026
This branch consolidates several [PRs](https://github.com/Next-Flip/Momentum-Firmware/issues?q=is:pr+is:open+author:956MB) (UI fixes and additions) into a usable build. Flash space is extremely low as is, and excluding some weather stations and other SubGHz protocols is necessary to be able to fit these PRs and allow for further changes. The ones that have been *temporarily* excluded are hopefully uncommon or legacy protocols that won't be missed if you're using this branch. The point of this branch being primarily UI features.

* #327
* #344
* #354
* #385

Excluded protocols:

* Uncommon/Regional gate, garage, and alarm protocols: `nero_sketch`, `nero_radio`, `ido`, `kia`, `power_smart`, `doitrand`, `ansonic`, `smc5326`, `linear_delta3`, `dooya`, `kinggates_stylo_4k`, `dickert_mahs`, `hay21`, `feron`, `elplast`, `schrader_gg4`, `tpms_generic`.
* Obscure weather-station protocols: `solight_te44`, `infactory`, `acurite_986`, `auriol_ahfl`, `auriol_hg0601a`, `emos_e601x`, `gt_wt_02`, `gt_wt_03`, `kedsum_th`, `nexus_th`, `oregon_v1`, `thermopro_tx4`, `tx_8300`, `vauno_en8822c`, `wendox_w6726`.
@956MB 956MB mentioned this pull request Jan 2, 2026
@956MB 956MB moved this from In Progress to Done in Momentum Progress Tracker Jan 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant