Skip to content

Add SVG corner logo support and custom PNG logo uploads - #214

Open
lanes100 wants to merge 2 commits into
netbymatt:mainfrom
lanes100:feature/logo-corner-svg
Open

lanes100 wants to merge 2 commits into
netbymatt:mainfrom
lanes100:feature/logo-corner-svg

Conversation

@lanes100

Copy link
Copy Markdown
Contributor

Summary

Replaces logo-corner.png usage with a new SVG version of the Weather Star 4000+ corner logo and adds support for uploading custom PNG corner logos through the admin interface.

Changes

  • Adds server/images/logos/logo-corner.svg
  • Updates header and radar partials to use the SVG logo by default
  • Preserves the existing rendered size at 85x67
  • Uses the original Weather Channel logo geometry as the template for outline proportions
  • Keeps the project’s original blue color and Weather Star 4000+ text treatment
  • Updates compiled weather display CSS for explicit SVG sizing
  • Adds support for uploading custom PNG corner logos
  • Updates logo handling logic to support both SVG defaults and uploaded PNG overrides

Testing

  • Verified the SVG renders correctly in the app at the intended corner-logo size
  • Confirmed header and radar views use the new SVG asset by default
  • Tested PNG upload functionality and verified uploaded logos display correctly

Screenshots

SVG in 16:9

image

Uploaded PNG in 16:9

image

PNG upload UI

image image

@lanes100

lanes100 commented May 15, 2026

Copy link
Copy Markdown
Contributor Author

I also want to note here that I did test the PNG upload with abnormally sized images as well and none of those threw any errors or looked weird. Everything works as intended from my testing.

@netbymatt

Copy link
Copy Markdown
Owner

Thank you for replacing the logo and for the clever idea of using LocalStorage to hold a user-defined logo. I might use this technique in the future for other customizations.

Also, the screenshot of the new settings is very appreciated.

The timing of this update was a little unfortunate and your changes and the recent 7.0 updates have created some conflicts. Would you be able to work through the conflicts prior to a merge?

@lanes100

Copy link
Copy Markdown
Contributor Author

Thank you for replacing the logo and for the clever idea of using LocalStorage to hold a user-defined logo. I might use this technique in the future for other customizations.

Also, the screenshot of the new settings is very appreciated.

The timing of this update was a little unfortunate and your changes and the recent 7.0 updates have created some conflicts. Would you be able to work through the conflicts prior to a merge?

Sure no worries. I'll work through the merge conflicts here in the next day or two.

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