Fixed: clickable link block (#6444)

* <span><a> => only <a>

* update frss.css

* update themes

11 files changed, 20 insertions, 24 deletions

diff --git a/app/views/helpers/index/normal/entry_header.phtml b/app/views/helpers/index/normal/entry_header.phtml
index ebac7f46b..0db375309 100644
--- a/app/views/helpers/index/normal/entry_header.phtml
+++ b/app/views/helpers/index/normal/entry_header.phtml
@@ -52,7 +52,7 @@
 	?></li><?php
 	endif; ?>
 	<li class="item titleAuthorSummaryDate">
-	<span class="item-element title<?= (($topline_thumbnail !== 'none') || $topline_summary) ? ' multiline' : '' ?>" dir="auto"><a target="_blank" rel="noreferrer" href="<?= $this->entry->link() ?>"><?= $this->entry->title() ?><?php
+		<a target="_blank" rel="noreferrer" href="<?= $this->entry->link() ?>" class="item-element title<?= (($topline_thumbnail !== 'none') || $topline_summary) ? ' multiline' : '' ?>" dir="auto"><?= $this->entry->title() ?><?php
 		if ($topline_display_authors):
 			?><span class="author"><?php
 			$authors = $this->entry->authors();
@@ -65,7 +65,7 @@
 			}
 			?></span><?php
 		endif;
-	?></a></span>
+		?></a>
 	<?php
 	if ($topline_summary):
 		?><div class="summary"><?= trim(mb_substr(strip_tags($this->entry->content(false)), 0, 500, 'UTF-8')) ?></div><?php
diff --git a/p/themes/Alternative-Dark/adark.css b/p/themes/Alternative-Dark/adark.css
index 2743de88b..1bc7a5097 100644
--- a/p/themes/Alternative-Dark/adark.css
+++ b/p/themes/Alternative-Dark/adark.css
@@ -759,7 +759,7 @@ kbd {
 	background: var(--background-color-hover);
 }
 
-.flux .flux_header .item .title a {
+.flux .flux_header .item .title {
 	color: var(--font-color-light);
 }
 
diff --git a/p/themes/Alternative-Dark/adark.rtl.css b/p/themes/Alternative-Dark/adark.rtl.css
index b66fae387..f618560a9 100644
--- a/p/themes/Alternative-Dark/adark.rtl.css
+++ b/p/themes/Alternative-Dark/adark.rtl.css
@@ -759,7 +759,7 @@ kbd {
 	background: var(--background-color-hover);
 }
 
-.flux .flux_header .item .title a {
+.flux .flux_header .item .title {
 	color: var(--font-color-light);
 }
 
diff --git a/p/themes/Dark/dark.css b/p/themes/Dark/dark.css
index 95ae5193b..4097e826c 100644
--- a/p/themes/Dark/dark.css
+++ b/p/themes/Dark/dark.css
@@ -452,7 +452,7 @@ button.as-link[disabled] {
 	background-color: var(--dark-background-color1);
 }
 
-.flux .flux_header .item .title a {
+.flux .flux_header .item .title {
 	color: var(--dark-font-color8);
 }
 
diff --git a/p/themes/Dark/dark.rtl.css b/p/themes/Dark/dark.rtl.css
index 670fde496..6a4cb4de2 100644
--- a/p/themes/Dark/dark.rtl.css
+++ b/p/themes/Dark/dark.rtl.css
@@ -452,7 +452,7 @@ button.as-link[disabled] {
 	background-color: var(--dark-background-color1);
 }
 
-.flux .flux_header .item .title a {
+.flux .flux_header .item .title {
 	color: var(--dark-font-color8);
 }
 
diff --git a/p/themes/Nord/nord.css b/p/themes/Nord/nord.css
index ef7b58aaf..8687ecfe1 100644
--- a/p/themes/Nord/nord.css
+++ b/p/themes/Nord/nord.css
@@ -810,7 +810,7 @@ li.item.active {
 	transition: .3s;
 }
 
-.flux .flux_header .item .title a {
+.flux .flux_header .item .title {
 	color: var(--text-default);
 }
 
diff --git a/p/themes/Nord/nord.rtl.css b/p/themes/Nord/nord.rtl.css
index df383289b..129feb266 100644
--- a/p/themes/Nord/nord.rtl.css
+++ b/p/themes/Nord/nord.rtl.css
@@ -810,7 +810,7 @@ li.item.active {
 	transition: .3s;
 }
 
-.flux .flux_header .item .title a {
+.flux .flux_header .item .title {
 	color: var(--text-default);
 }
 
diff --git a/p/themes/Pafat/pafat.css b/p/themes/Pafat/pafat.css
index 072a52e46..f234194ff 100644
--- a/p/themes/Pafat/pafat.css
+++ b/p/themes/Pafat/pafat.css
@@ -803,7 +803,7 @@ a.signin {
 	background-color: var(--background-color-grey-hover);
 }
 
-.flux .flux_header:hover .title a {
+.flux .flux_header:hover .title {
 	color: var(--font-color-hover);
 }
 
diff --git a/p/themes/Pafat/pafat.rtl.css b/p/themes/Pafat/pafat.rtl.css
index daa7ea233..8e8d5ca59 100644
--- a/p/themes/Pafat/pafat.rtl.css
+++ b/p/themes/Pafat/pafat.rtl.css
@@ -803,7 +803,7 @@ a.signin {
 	background-color: var(--background-color-grey-hover);
 }
 
-.flux .flux_header:hover .title a {
+.flux .flux_header:hover .title {
 	color: var(--font-color-hover);
 }
 
diff --git a/p/themes/base-theme/frss.css b/p/themes/base-theme/frss.css
index 1f010cc31..ff1be45b9 100644
--- a/p/themes/base-theme/frss.css
+++ b/p/themes/base-theme/frss.css
@@ -1294,11 +1294,6 @@ a.website:hover .favicon {
 	vertical-align: top;
 }
 
-.flux .flux_header .item .title a {
-	color: var(--frss-font-color-dark);
-	text-decoration: none;
-}
-
 .flux .flux_header .item.thumbnail {
 	line-height: 0;
 	padding: var(--frss-padding-top-bottom) var(--frss-padding-flux-items);
@@ -1353,13 +1348,16 @@ a.website:hover .favicon {
 }
 
 .flux .flux_header .item .title {
+	display: inline-block;
+	max-width: 100%;
+	min-width: calc(100% - 155px);
+	color: var(--frss-font-color-dark);
 	position: absolute;
 	top: 0;
 	overflow: hidden;
 	text-overflow: ellipsis;
-	display: inline-block;
 	box-sizing: border-box;
-	max-width: 100%;
+	text-decoration: none;
 }
 
 .flux .flux_header .item .title:has(~.date) {
diff --git a/p/themes/base-theme/frss.rtl.css b/p/themes/base-theme/frss.rtl.css
index 3085529fc..b2a2b9265 100644
--- a/p/themes/base-theme/frss.rtl.css
+++ b/p/themes/base-theme/frss.rtl.css
@@ -1294,11 +1294,6 @@ a.website:hover .favicon {
 	vertical-align: top;
 }
 
-.flux .flux_header .item .title a {
-	color: var(--frss-font-color-dark);
-	text-decoration: none;
-}
-
 .flux .flux_header .item.thumbnail {
 	line-height: 0;
 	padding: var(--frss-padding-top-bottom) var(--frss-padding-flux-items);
@@ -1353,13 +1348,16 @@ a.website:hover .favicon {
 }
 
 .flux .flux_header .item .title {
+	display: inline-block;
+	max-width: 100%;
+	min-width: calc(100% - 155px);
+	color: var(--frss-font-color-dark);
 	position: absolute;
 	top: 0;
 	overflow: hidden;
 	text-overflow: ellipsis;
-	display: inline-block;
 	box-sizing: border-box;
-	max-width: 100%;
+	text-decoration: none;
 }
 
 .flux .flux_header .item .title:has(~.date) {