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

Sonarr : Add-on : sonarr-text-logo #415

Merged
merged 7 commits into from
Aug 26, 2022
Merged

Sonarr : Add-on : sonarr-text-logo #415

merged 7 commits into from
Aug 26, 2022

Conversation

neoestremi
Copy link
Contributor

theme-park.dev


  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

Adds a custom logo for Sonarr.

Benefits of this PR and context:

Have the possibility to make Sonarr look even more like Radarr

How Has This Been Tested?

Editing the html code and css inside browser with the values used by Radarr and the custom logo, it matched perfectly.

Source / References:

css based on the radarr-4k-logo

@GilbN
Copy link
Collaborator

GilbN commented Aug 26, 2022

Thanks for PR! Looks good. I removed the replacement of the logo when on mobile as that broke the alignment of search.

@GilbN GilbN merged commit d2d413d into themepark-dev:develop Aug 26, 2022
@neoestremi
Copy link
Contributor Author

Thank you. I see you changed the size too. I made the logo with the same size as radarr. And used the size used in the site too, did you check it was the same ? The idea was to have them being “replaced” when switching tabs.
Again, thank you for adding it :)

@GilbN
Copy link
Collaborator

GilbN commented Aug 27, 2022

Yeah. I noticed that it got cut off at the end when using 120px width.

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