Skip to content

Conversation

@stefan123t
Copy link
Collaborator

updated the logo with a proper logarithmic spiral cable

leave 25 pixels, i.e. 12.5 top and bottom
added a 64x64 pixel version of the logo for OLED 128x64 and Nokia LCD displays. 
Reduced structure to 64x64 to conserve half of the PROGMEM space instead of 128x64 pixel size.
May need to be displayed centered on display.
15 px top and 15 px bottom white border added for Knickohr
@lumapu
Copy link
Owner

lumapu commented May 29, 2025

The 64x64 graphic is currently not implemented - am I right?

@lumapu
Copy link
Owner

lumapu commented May 29, 2025

wouldn't it be better to align the logo in ePaper centered as well and remove the 25px white border from the image data? Memory is costly, therefore I would prefer to store only necessary data.

@stefan123t
Copy link
Collaborator Author

@lumapu yes, these are both valid points.
I can remove the border around the larger 200x200 logo, that would even save some more space left and right too ...
Someone who does the Display stuff would need to align the two versions on the different displays, i.e. OLED, Nokia LCD and ePaper's.
Cheers,
Stefan

As requested by @lumapu I have reduced both logo sizes.
I have added the display size these logos fit and the required left/right and top/bottom padding in a comment each.
You will have to replace all occurrences of logo with logo_spiral_140x150 or implement logo_spiral_52x56 for the OLED and Nokia LCD Displays.
@stefan123t
Copy link
Collaborator Author

@lumapu I have prepared two Versions for the different displays:

  • for OLED, Nokia LCD: logo_spiral_52x56.png
    • fits 64x64 or 64x128, etc.
    • padding top/bottom: 4 px
    • padding left/right 6 px
      logo_spiral_52x56
  • for ePaper logo_spiral_140x150.png
    • fits 200x200
    • padding top/botton: 25 px
    • padding left/right: 30 px
      logo_spiral_140x150

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.

3 participants