Skip to content

Fixing CSS Issues in sabnzbd-base.css#680

Merged
GilbN merged 1 commit intothemepark-dev:developfrom
zdhoward:develop
Apr 21, 2025
Merged

Fixing CSS Issues in sabnzbd-base.css#680
GilbN merged 1 commit intothemepark-dev:developfrom
zdhoward:develop

Conversation

@zdhoward
Copy link
Copy Markdown
Contributor

theme-park.dev


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

  • PR's are done against the develop branch.


Bug fixes

  • Before images:
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image

  • After images:
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image
    image

image
image

image
image

Description:

I went through the sabnzbd-base.css file and did my best to use what already existed to fix up css issues all over.
Tested mainly using hotline.css

This is also my first contribution to this project so if anything here isn't done correctly, just let me know and I'll be happy to fix it.

Benefits of this PR and context:

Fixes existing base css issues for the sabnzbd app

How Has This Been Tested?

I am using brave browser and did most of my work in the inspector, under Source with the sabnzbd-base.css file.
Then I put it in my theme-park docker container to let my app grab it using docker mods.

Testing was just looking through each page and seeing if any elements weren't using values from the theme.

Should only effect other themes on sabnzbd, but as long as the colors are set up in the same way as hotline.css then it should work well. Spot checked a couple others like hotpink and plex and it is all looking good.

Source / References:

@GilbN
Copy link
Copy Markdown
Collaborator

GilbN commented Apr 21, 2025

Thanks for the fix! Looks great!

@GilbN GilbN merged commit de7a298 into themepark-dev:develop Apr 21, 2025
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