themepark-dev / theme.park

A collection of themes/skins for 50 selfhosted apps!
https://theme-park.dev
MIT License
2.43k stars 799 forks source link

Radarr, & Prowlarr Theme Issues #512

Closed ALameLlama closed 1 year ago

ALameLlama commented 1 year ago

Describe the bug Looks like a similar issue has happened with the where the sidebar and search/top bar aren't getting themed correctly and say the default grey. I've tried a few different themes and they all have the same issue

Theme Install Method (Don't skip) https://plugins.traefik.io/plugins/62947288ffc0cd18356a97c1/theme-park Using these theme with treafik to rewrite the headers.

    radarr-themepark:
      plugin:
        traefik-themepark:
          app: radarr
          theme: hotline
    prowlarr-themepark:
      plugin:
        traefik-themepark:
          app: prowlarr
          theme: hotline

Application Info (Don't skip) Radarr: 4.7.5.7809 (linuxserver) - Lastest With this in the header:

Prowlarr: 1.7.4.3769 (linuxserver) - Lastest With this in the header:

To Reproduce Load Web Browser

Expected behavior Theme should update the side and top bar like it does in the examples.

Screenshots Sonarr (working) image

Radarr: image image

GilbN commented 1 year ago

Pretty sure it's because the plugin doesn't subfilter the body tag. Radarr etc needs subfiltering on the body tag. You should contact the dev of the plugin.

GilbN commented 1 year ago

https://github.com/packruler/traefik-themepark/blob/2d354c89713d4591fea98358c261e4f1fc8239dc/themepark.go#L84

GilbN commented 1 year ago

https://github.com/packruler/traefik-themepark/issues/10

ALameLlama commented 1 year ago

@GilbN thanks for that and sorry for bugging you.