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

Splash resize #2353

Merged
merged 4 commits into from
Nov 9, 2024
Merged

Splash resize #2353

merged 4 commits into from
Nov 9, 2024

Conversation

gullradriel
Copy link
Member

@gullradriel gullradriel commented Nov 9, 2024

The Y position of the splash screen bitmap was not right, by 1 pixel.
The X offset was badly set, and anyway is not used. I set it back to zero, so if at some point someone is fixing the display bitmap, splash screen will still be ok.

The definite displayed size of a splash is 240x304 pixels.

Tests have shown that:

  • 240 pixel wide is mandatory
  • height can vary from 1 to 304 pixels
  • more than 304 pixels in height will result in splash screen truncated (no dynamic resize, only the first 304 pixels are displayed)

=> the default splash bitmaps have been resized to maximum displayed size

=> wiki page have been updated : https://github.com/portapack-mayhem/mayhem-firmware/wiki/Create-a-custom-splash-screen

Copy link
Member

@htotoo htotoo left a comment

Choose a reason for hiding this comment

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

Lgtm

@gullradriel gullradriel merged commit 458a663 into portapack-mayhem:next Nov 9, 2024
3 checks passed
@gullradriel gullradriel deleted the splash-resize branch November 9, 2024 14:12
htotoo pushed a commit to htotoo/portapack-mayhem that referenced this pull request Nov 11, 2024
* resized default splash screens to maximum displayed size
* fix splash screen X,Y position to 0,16
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