LPS-140315 Only add aria attributes when a coverImageCaption exist

boton · Oct 8, 2021 · 6a1cb30 · 6a1cb30
1 parent bb7386e
commit 6a1cb30
Show file tree

Hide file tree

Showing 5 changed files with 30 additions and 5 deletions.
diff --git a/modules/apps/blogs/blogs-web/src/main/resources/META-INF/resources/blogs/asset/abstract.jsp b/modules/apps/blogs/blogs-web/src/main/resources/META-INF/resources/blogs/asset/abstract.jsp
@@ -27,7 +27,12 @@ BlogsEntry entry = (BlogsEntry)request.getAttribute(WebKeys.BLOGS_ENTRY);
 	%>
 
 	<c:if test="<%= Validator.isNotNull(coverImageURL) %>">
-		<div aria-label="<%= HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption())) %>" class="aspect-ratio aspect-ratio-8-to-3 aspect-ratio-bg-cover cover-image mb-4" role="img" style="background-image: url(<%= coverImageURL %>);"></div>
+
+		<%
+		String coverImageCaption = HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption()));
+		%>
+
+		<div <c:if test="<%= Validator.isNotNull(coverImageCaption) %>"> aria-label="<%= coverImageCaption %>" role="img" </c:if> class="aspect-ratio aspect-ratio-8-to-3 aspect-ratio-bg-cover cover-image mb-4" style="background-image: url(<%= coverImageURL %>);"></div>
 	</c:if>
 
 	<%

diff --git a/...s/apps/blogs/blogs-web/src/main/resources/META-INF/resources/blogs/asset/full_content.jsp b/...s/apps/blogs/blogs-web/src/main/resources/META-INF/resources/blogs/asset/full_content.jsp
@@ -40,7 +40,12 @@ String entryTitle = BlogsEntryUtil.getDisplayTitle(resourceBundle, entry);
 				%>
 
 				<c:if test="<%= Validator.isNotNull(coverImageURL) %>">
-					<div aria-label="<%= HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption())) %>" class="aspect-ratio aspect-ratio-8-to-3 aspect-ratio-bg-cover cover-image" role="img" style="background-image: url(<%= coverImageURL %>);"></div>
+
+					<%
+					String coverImageCaption = HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption()));
+					%>
+
+					<div <c:if test="<%= Validator.isNotNull(coverImageCaption) %>"> aria-label="<%= coverImageCaption %>" role="img" </c:if> class="aspect-ratio aspect-ratio-8-to-3 aspect-ratio-bg-cover cover-image" style="background-image: url(<%= coverImageURL %>);"></div>
 				</c:if>
 
 				<%= entry.getContent() %>

diff --git a/...blogs-web/src/main/resources/META-INF/resources/blogs/info/item/renderer/full_content.jsp b/...blogs-web/src/main/resources/META-INF/resources/blogs/info/item/renderer/full_content.jsp
@@ -31,7 +31,12 @@ String entryTitle = BlogsEntryUtil.getDisplayTitle(resourceBundle, entry);
 			%>
 
 			<c:if test="<%= Validator.isNotNull(coverImageURL) %>">
-				<div aria-label="<%= HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption())) %>" class="aspect-ratio aspect-ratio-8-to-3 aspect-ratio-bg-cover cover-image" role="img" style="background-image: url(<%= coverImageURL %>);"></div>
+
+				<%
+				String coverImageCaption = HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption()));
+				%>
+
+				<div <c:if test="<%= Validator.isNotNull(coverImageCaption) %>"> aria-label="<%= coverImageCaption %>" role="img" </c:if> class="aspect-ratio aspect-ratio-8-to-3 aspect-ratio-bg-cover cover-image" style="background-image: url(<%= coverImageURL %>);"></div>
 			</c:if>
 
 			<%= entry.getContent() %>

diff --git a/...s/apps/blogs/blogs-web/src/main/resources/META-INF/resources/blogs/view_entry_content.jsp b/...s/apps/blogs/blogs-web/src/main/resources/META-INF/resources/blogs/view_entry_content.jsp
@@ -140,7 +140,12 @@ BlogsPortletInstanceConfiguration blogsPortletInstanceConfiguration = BlogsPortl
 
 				<c:if test="<%= Validator.isNotNull(coverImageURL) %>">
 					<a href="<%= viewEntryURL.toString() %>">
-						<div aria-label="<%= HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption())) %>" class="aspect-ratio aspect-ratio-8-to-3 aspect-ratio-bg-cover cover-image" role="img" style="background-image: url(<%= coverImageURL %>);"></div>
+
+						<%
+						String coverImageCaption = HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption()));
+						%>
+
+						<div <c:if test="<%= Validator.isNotNull(coverImageCaption) %>"> aria-label="<%= coverImageCaption %>" role="img" </c:if> class="aspect-ratio aspect-ratio-8-to-3 aspect-ratio-bg-cover cover-image" style="background-image: url(<%= coverImageURL %>);"></div>
 					</a>
 				</c:if>
 

diff --git a/...blogs/blogs-web/src/main/resources/META-INF/resources/blogs/view_entry_content_detail.jsp b/...blogs/blogs-web/src/main/resources/META-INF/resources/blogs/view_entry_content_detail.jsp
@@ -178,7 +178,12 @@ BlogsPortletInstanceConfiguration blogsPortletInstanceConfiguration = BlogsPortl
 		%>
 
 		<c:if test="<%= Validator.isNotNull(coverImageURL) %>">
-			<div aria-label="<%= HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption())) %>" class="aspect-ratio aspect-ratio-bg-cover cover-image" role="img" style="background-image: url(<%= coverImageURL %>);"></div>
+
+			<%
+			String coverImageCaption = HtmlUtil.escapeAttribute(HtmlUtil.stripHtml(entry.getCoverImageCaption()));
+			%>
+
+			<div <c:if test="<%= Validator.isNotNull(coverImageCaption) %>"> aria-label="<%= coverImageCaption %>" role="img" </c:if> class="aspect-ratio aspect-ratio-8-to-3 aspect-ratio-bg-cover cover-image" style="background-image: url(<%= coverImageURL %>);"></div>
 		</c:if>
 
 		<!-- text resume -->