From 1ca20c5529e52e4e058daab0ea3248b257612821 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Fromme?= <fromme@dreipunktnull.com>
Date: Thu, 31 Aug 2023 11:03:11 +0200
Subject: [PATCH] Task: Cleanup itemscope attributes of news articles

---
 .../Private/Layouts/News/Detail.html          |  4 +-
 .../Private/Partials/News/List/Item.html      |  8 +--
 .../Private/Templates/News/News/Detail.html   | 60 ++++++++++---------
 3 files changed, 36 insertions(+), 36 deletions(-)

diff --git a/public/typo3conf/ext/ep_theme/Resources/Private/Layouts/News/Detail.html b/public/typo3conf/ext/ep_theme/Resources/Private/Layouts/News/Detail.html
index 6a3dab78..dbdac17d 100644
--- a/public/typo3conf/ext/ep_theme/Resources/Private/Layouts/News/Detail.html
+++ b/public/typo3conf/ext/ep_theme/Resources/Private/Layouts/News/Detail.html
@@ -2,8 +2,6 @@
 	  xmlns:n="http://typo3.org/ns/GeorgRinger/News/ViewHelpers"
 	  data-namespace-typo3-fluid="true">
 
-<div itemscope="itemscope" itemtype="http://schema.org/Article">
-    <f:render section="content" />
-</div>
+<f:render section="content" />
 
 </html>
diff --git a/public/typo3conf/ext/ep_theme/Resources/Private/Partials/News/List/Item.html b/public/typo3conf/ext/ep_theme/Resources/Private/Partials/News/List/Item.html
index f1ee3a76..c8d284bc 100644
--- a/public/typo3conf/ext/ep_theme/Resources/Private/Partials/News/List/Item.html
+++ b/public/typo3conf/ext/ep_theme/Resources/Private/Partials/News/List/Item.html
@@ -2,7 +2,7 @@
 	  xmlns:n="http://typo3.org/ns/GeorgRinger/News/ViewHelpers"
 	  data-namespace-typo3-fluid="true">
 
-    <div class="mb-8" itemscope="itemscope" itemtype="http://schema.org/Article">
+    <div class="mb-8">
         <n:excludeDisplayedNews newsItem="{newsItem}"/>
         <div class="flex flex-col md:flex-row md:space-x-8">
             <n:link newsItem="{newsItem}" settings="{settings}" title="{newsItem.title}" class="mb-4 md:w-1/2">
@@ -32,12 +32,12 @@
             <div class="md:flex-1">
                 <h3 class="blog-entry__headline">
                     <n:link newsItem="{newsItem}" settings="{settings}" title="{newsItem.title}">
-                        <span itemprop="headline">{newsItem.title}</span>
+                        {newsItem.title}
                     </n:link>
                 </h3>
                 <p class="text-sm italic mb-4">
                     <f:if condition="{newsItem.author}">{newsItem.author} am </f:if>
-                    <time itemprop="datePublished" datetime="{f:format.date(date:newsItem.datetime, format:'Y-m-d')}"><f:format.date format="{f:translate(key:'dateFormat')}">{newsItem.datetime}</f:format.date></time>
+                    <time datetime="{f:format.date(date:newsItem.datetime, format:'Y-m-d')}"><f:format.date format="{f:translate(key:'dateFormat')}">{newsItem.datetime}</f:format.date></time>
                     <f:if condition="{newsItem.tags}"> in
                         <f:for each="{newsItem.tags}" as="tag" iteration="iteration">
                             <f:link.page title="{tag.title}" pageUid="{settings.tagListPageUid}" additionalParams="{tx_news_pi1:{overwriteDemand:{tags: tag}}}">
@@ -48,7 +48,7 @@
                     </f:if>
                 </p>
                 <n:removeMediaTags>
-                    <div class="mb-4" itemprop="description">
+                    <div class="mb-4">
                         <f:if condition="{newsItem.teaser}">
                             <f:then>
                                 {newsItem.teaser -> f:format.crop(maxCharacters: '{settings.cropMaxCharacters}', respectWordBoundaries:'1') -> f:format.html()}
diff --git a/public/typo3conf/ext/ep_theme/Resources/Private/Templates/News/News/Detail.html b/public/typo3conf/ext/ep_theme/Resources/Private/Templates/News/News/Detail.html
index 1455ce99..a600b036 100644
--- a/public/typo3conf/ext/ep_theme/Resources/Private/Templates/News/News/Detail.html
+++ b/public/typo3conf/ext/ep_theme/Resources/Private/Templates/News/News/Detail.html
@@ -23,36 +23,38 @@
                     </f:if>
                     <f:render partial="Detail/Opengraph" arguments="{newsItem: newsItem, settings:settings}" />
                 </n:format.nothing>
-                <h1 class="headline headline--primary headline--underlined" itemprop="headline">
-                    {newsItem.title}
-                </h1>
-                <p class="text-sm italic mb-4">
-                    <f:if condition="{newsItem.author}">{newsItem.author} am </f:if>
-                    <time itemprop="datePublished" datetime="{f:format.date(date:newsItem.datetime, format:'Y-m-d')}"><f:format.date format="{f:translate(key:'dateFormat')}">{newsItem.datetime}</f:format.date></time>
-                    <f:if condition="{newsItem.tags}"> in
-                        <f:for each="{newsItem.tags}" as="tag" iteration="iteration">
-                            <f:link.page title="{tag.title}" pageUid="{settings.tagListPageUid}" additionalParams="{tx_news_pi1:{overwriteDemand:{tags: tag}}}">
-                                {tag.title}
-                            </f:link.page>
-                            {f:if(condition: iteration.isLast, else: ', ')}
-                        </f:for>
-                    </f:if>
-                </p>
-                <f:if condition="{newsItem.teaser}">
-                    <p class="text-lg mb-4">
-                        <f:format.nl2br>{newsItem.teaser}</f:format.nl2br>
-                    </p>
-                </f:if>
-                <div data-rte-content>
-                    <n:renderMedia news="{newsItem}" imgClass="block w-full h-auto mb-4" videoClass="video-wrapper" audioClass="audio-wrapper">
-                        <f:render partial="Detail/MediaContainer" arguments="{media: newsItem.media, settings:settings, id: newsItem.uid}" />
-                        <f:format.html>{newsItem.bodytext}</f:format.html>
-                        <f:if condition="{newsItem.contentElements}">
-                            <div class="flex flex-col space-y-4">
-                                <f:cObject typoscriptObjectPath="lib.tx_news.contentElementRendering">{newsItem.contentElementIdList}</f:cObject>
-                            </div>
+                <div itemscope="itemscope" itemtype="http://schema.org/Article">
+                    <h1 class="headline headline--primary headline--underlined" itemprop="headline">
+                        {newsItem.title}
+                    </h1>
+                    <p class="text-sm italic mb-4">
+                        <f:if condition="{newsItem.author}">{newsItem.author} am </f:if>
+                        <time itemprop="datePublished" datetime="{f:format.date(date:newsItem.datetime, format:'Y-m-d')}"><f:format.date format="{f:translate(key:'dateFormat')}">{newsItem.datetime}</f:format.date></time>
+                        <f:if condition="{newsItem.tags}"> in
+                            <f:for each="{newsItem.tags}" as="tag" iteration="iteration">
+                                <f:link.page title="{tag.title}" pageUid="{settings.tagListPageUid}" additionalParams="{tx_news_pi1:{overwriteDemand:{tags: tag}}}">
+                                    {tag.title}
+                                </f:link.page>
+                                {f:if(condition: iteration.isLast, else: ', ')}
+                            </f:for>
                         </f:if>
-                    </n:renderMedia>
+                    </p>
+                    <f:if condition="{newsItem.teaser}">
+                        <p class="text-lg mb-4">
+                            <f:format.nl2br>{newsItem.teaser}</f:format.nl2br>
+                        </p>
+                    </f:if>
+                    <div data-rte-content>
+                        <n:renderMedia news="{newsItem}" imgClass="block w-full h-auto mb-4" videoClass="video-wrapper" audioClass="audio-wrapper">
+                            <f:render partial="Detail/MediaContainer" arguments="{media: newsItem.media, settings:settings, id: newsItem.uid}" />
+                            <f:format.html>{newsItem.bodytext}</f:format.html>
+                            <f:if condition="{newsItem.contentElements}">
+                                <div class="flex flex-col space-y-4">
+                                    <f:cObject typoscriptObjectPath="lib.tx_news.contentElementRendering">{newsItem.contentElementIdList}</f:cObject>
+                                </div>
+                            </f:if>
+                        </n:renderMedia>
+                    </div>
                 </div>
                 <f:if condition="{newsItem.relatedFiles}">
                     <h3 class="headline headline--underlined">