Conversation
Fix transmission
The recent Ombi redesign rebuilt the login page, requests list, TV request grid + requests panel, media details hero, discover/detailed cards, cast carousel, social icons, the discover hero banner, and the sidebar + top bar with brand-new class names. None of those new selectors were styled by theme.park, so users on current Ombi were seeing Ombi's stock $ombi-active palette instead of their selected theme. Add overrides that map the new selectors onto theme.park's CSS variables (accent, text, transparency layers, button colors, modal background) so the existing themes work again on current Ombi. Covered: - Login: .login-panel, .brand-title, .input-wrapper(--focused/--error), .login-input, .input-icon (focused state), .toggle-password, .checkmark + .checkmark::after, .custom-checkbox, .forgot-link, .btn-submit, .btn-login--primary (Plex/Emby/Jellyfin brand buttons intentionally left untouched). - Hero banner: .hero-banner, .hero-overlay, .hero-title, .hero-cta, .hero-indicator(.active). - Media details: .details-hero, .hero-backdrop-overlay, .meta-badge(--accent), .action-btn(--primary/--secondary), .info-panel-modern, .info-grid/-item/-label/-value, .streaming-label/-logo, .genre-tag. - Cast / social: .cast-card, .cast-profile-img (+ accent hover), .cast-name, .cast-character, .social-link, .social-divider, .admin-cog-btn. - Discover cards: .detailed-card, .card-inner, .card-status-badge, .approve-btn, .deny-btn, .request-btn, .filter-toggle-group, .filter-active, .genre-chip. - Requests list: .requests-page, .tab-bar/.tab-item.active, .chip(.active), .request-card(.selected), .poster-overlay, .btn-detail, .btn-options, .fab-button, .no-poster, .empty-state. - TV requests panel: .request-header, .request-season-chip(.active), .req-ep-*, .expand-icon. - TV request grid: .quick-action-btn, .season-chip(.active), .season-panel, .episode-count-badge, .season-progress-bar, .episode-row, .sticky-bar, .request-selected-btn, .section-divider. - Sidebar + top bar (rebuild beyond the linked commits): .sidebar-brand .brand-link (was .application-name), .nav-item, .nav-item--active (was .active-list-item), .nav-icon, .nav-indicator, .nav-section-label, .nav-divider, .sidebar-footer, .top-bar (was .top-bar-container), .hamburger-btn, .top-bar .chip / .chip.active (scoped so it doesn't clash with the requests-list .chip), .profile-link, .profile-username, .profile-avatar. Old Material-UI rules are kept so users on the previous Ombi release (which still ships the old DOM) aren't affected. Verified by building Ombi develop locally and using Playwright to read computed styles + capture screenshots on the live build with both the dark and plex themes injected. Brand link, active nav indicator, hero CTA, top-bar chip active state, and profile avatar hover ring all resolve to the selected theme's accent (rgb(170,170,170) for dark, rgb(229,160,13) for plex).
ombi: support the UI redesign
Fix Seerr background selector after update
There was a problem hiding this comment.
Pull request overview
This PR updates several theme.park base styles and a Transmission docker-mod init script to keep theming compatible with upstream UI/DOM changes across multiple apps (SABnzbd, Ombi v5, Overseerr, Uptime-Kuma 2.1.0, Transmission).
Changes:
- Transmission docker-mod: update the target
index.htmlpath from.../web/to.../public_html/for stylesheet injection. - Uptime-Kuma base: remove global
canvas { filter: invert(1) }to prevent inverted heartbeat colors (Issue #714). - App CSS updates: SABnzbd graph color uses accent color; Overseerr background selector adjusted; Ombi base adds large v5 redesign override block.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| docker-mods/transmission/root/etc/s6-overlay/s6-rc.d/init-mod-themepark/run | Updates Transmission web root path for CSS injection (but currently duplicates injection commands). |
| docker-mods/transmission/root/etc/cont-init.d/98-themepark | Updates Transmission web root path for CSS injection during container init. |
| css/base/uptime-kuma/uptime-kuma-base.css | Removes canvas invert filter to fix inverted heartbeat colors on newer Uptime-Kuma. |
| css/base/sabnzbd/sabnzbd-base.css | Sets SABnzbd peity graph fill/stroke to the configured accent color. |
| css/base/overseerr/overseerr-base.css | Adjusts background selector to match updated Overseerr DOM structure. |
| css/base/ombi/ombi-base.css | Adds extensive Ombi v5 redesign-specific overrides to restore theme styling. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+49
to
52
| sed -i "s/<\/head>/<link rel='stylesheet' href='${TP_SCHEME}:\/\/${TP_DOMAIN}\/css\/base\/transmission\/transmission-base.css'><\/head> /g" /usr/share/transmission/public_html/index.html | ||
| sed -i "s/<\/head>/<link rel='stylesheet' href='${TP_SCHEME}:\/\/${TP_DOMAIN}\/css\/${THEME_TYPE}\/${TP_THEME}.css'><\/head> /g" /usr/share/transmission/public_html/index.html | ||
| sed -i "s/<\/head>/<link rel='stylesheet' href='${TP_SCHEME}:\/\/${TP_DOMAIN}\/css\/base\/transmission\/transmission-base.css'><\/head> /g" /usr/share/transmission/public_html/index.html | ||
| sed -i "s/<\/head>/<link rel='stylesheet' href='${TP_SCHEME}:\/\/${TP_DOMAIN}\/css\/${THEME_TYPE}\/${TP_THEME}.css'><\/head> /g" /usr/share/transmission/public_html/index.html |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Set SABnzbd graph color to accent color
ombi-base: support the v5 UI redesign
Fix Seerr background selector after update
Fix file paths for HTML files in transmission
fix: Remove canvas filter from uptime-kuma-base.css #714