From 98eaed796bcf85cf4fac3eae744a31a83c1ed8c1 Mon Sep 17 00:00:00 2001 From: polybjorn Date: Fri, 15 May 2026 10:06:47 +0000 Subject: [PATCH] revert(themes): hide feed name at narrow viewports (#8838) Reverts #8801 (keeping its incidental blank-line cleanups) and #8815, restoring icon-only feed column on mobile (#8814). I filed the original PR thinking the icon-only behavior was an inconsistency bug versus the configurable `topline_website` setting, not realizing it was a deliberate mobile choice. The `clamp()` cap helped, but the column still takes space the title could use, and on phones icon-only is the better tradeoff. A distinct narrow-viewport setting for `topline_website` was raised as a future improvement. * Revert "fix(themes): cap website column at narrow when feed name is shown (#8815)" This reverts commit af6a5006382340865a1d68b0c3ff944fe20b8456. * Revert "fix(themes): show feed name at narrow widths when configured (#8801)" This reverts commit 9ff4aec368ba33e2ccab379f5fb1bdf4dae32da6. --------- Co-authored-by: FreshRSS Pi --- p/themes/base-theme/frss.css | 14 ++++++-------- p/themes/base-theme/frss.rtl.css | 14 ++++++-------- 2 files changed, 12 insertions(+), 16 deletions(-) diff --git a/p/themes/base-theme/frss.css b/p/themes/base-theme/frss.css index 4c256e795..bc667f7f2 100644 --- a/p/themes/base-theme/frss.css +++ b/p/themes/base-theme/frss.css @@ -2509,6 +2509,7 @@ html.slider-active { --frss-padding-flux-items: 0.5rem; } + .flux_header .item.website span, .item .date, .dropdown-menu > .no-mobile, .no-mobile, @@ -2598,6 +2599,10 @@ html.slider-active { list-style: none; } + .flux .flux_header.has-thumbnail.has-summary li.website .websiteName { + display: inline; + } + /** place all the elements in their respective template areas */ .flux .flux_header.has-thumbnail.has-summary li.website { grid-area: website; @@ -2828,17 +2833,10 @@ html.slider-active { padding: 0; } - /** Compensate for shrunk --frss-padding-flux-items so favicon cell stays 40px at narrow */ - .flux_header.websiteicon .item.website { + .flux_header .item.website { width: 40px; } - /** Cap website column when feed name is shown, to leave room for the title */ - .flux_header.websitename .item.website, - .flux_header.websitefull .item.website { - width: clamp(120px, 18vw, 150px); - } - .flux:not(.current):hover .item.title { position: relative; width: auto; diff --git a/p/themes/base-theme/frss.rtl.css b/p/themes/base-theme/frss.rtl.css index c216dfc6b..c7b859a2e 100644 --- a/p/themes/base-theme/frss.rtl.css +++ b/p/themes/base-theme/frss.rtl.css @@ -2509,6 +2509,7 @@ html.slider-active { --frss-padding-flux-items: 0.5rem; } + .flux_header .item.website span, .item .date, .dropdown-menu > .no-mobile, .no-mobile, @@ -2598,6 +2599,10 @@ html.slider-active { list-style: none; } + .flux .flux_header.has-thumbnail.has-summary li.website .websiteName { + display: inline; + } + /** place all the elements in their respective template areas */ .flux .flux_header.has-thumbnail.has-summary li.website { grid-area: website; @@ -2828,17 +2833,10 @@ html.slider-active { padding: 0; } - /** Compensate for shrunk --frss-padding-flux-items so favicon cell stays 40px at narrow */ - .flux_header.websiteicon .item.website { + .flux_header .item.website { width: 40px; } - /** Cap website column when feed name is shown, to leave room for the title */ - .flux_header.websitename .item.website, - .flux_header.websitefull .item.website { - width: clamp(120px, 18vw, 150px); - } - .flux:not(.current):hover .item.title { position: relative; width: auto;