Ver Fonte

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 <freshrss-pi@localhost>
polybjorn há 5 dias atrás
pai
commit
98eaed796b
2 ficheiros alterados com 12 adições e 16 exclusões
  1. 6 8
      p/themes/base-theme/frss.css
  2. 6 8
      p/themes/base-theme/frss.rtl.css

+ 6 - 8
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;

+ 6 - 8
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;