[FEATURE] Add rx_shariff option in news detail template. (#260)

t3kit · Sep 7, 2017 · 57cb425 · 57cb425
1 parent d439405
commit 57cb425
Show file tree

Hide file tree

Showing 8 changed files with 54 additions and 14 deletions.
diff --git a/Configuration/TypoScript/Library/plugin.rx_shariff.setupts b/Configuration/TypoScript/Library/plugin.rx_shariff.setupts
@@ -0,0 +1,17 @@
+tt_content.list.20.rxshariff_shariff {
+    settings {
+        enableBackend = 0
+        data {
+            lang = en
+            mail-body =
+            mail-subject =
+            mail-url = mailto:
+            media-url =
+            orientation = horizontal
+            referrer-track =
+            services =
+            theme = standard
+            twitter-via =
+        }
+    }
+}
diff --git a/Configuration/TypoScript/setup.txt b/Configuration/TypoScript/setup.txt
@@ -32,5 +32,10 @@
     <INCLUDE_TYPOSCRIPT: source="FILE:EXT:theme_t3kit/Resources/Private/Extensions/cs_seo/TypoScript/setup.ts">
 [global]
 
+[userFunc = TYPO3\CMS\Core\Utility\ExtensionManagementUtility::isLoaded('rx_shariff')]
+    <INCLUDE_TYPOSCRIPT: source="FILE:EXT:rx_shariff/Configuration/TypoScript/WithoutJQueryAndFontawesome/setup.txt">
+    <INCLUDE_TYPOSCRIPT: source="FILE:EXT:theme_t3kit/Configuration/TypoScript/Library/plugin.rx_shariff.setupts">
+[global]
+
 # Include setupTS files from fileadmin/Configuration/TypoScript
 <INCLUDE_TYPOSCRIPT: source="DIR:fileadmin/Configuration/TypoScript/" extensions="setupts">
diff --git a/Resources/Private/Extensions/News/Partials/Detail/Shariff.html b/Resources/Private/Extensions/News/Partials/Detail/Shariff.html
@@ -0,0 +1,2 @@
+{namespace rx=Reelworx\RxShariff\ViewHelper}
+<rx:shariff services="{settings.detail.shariffServices}" enableBackend="true"/>
diff --git a/Resources/Private/Extensions/News/Templates/News/Detail.html b/Resources/Private/Extensions/News/Templates/News/Detail.html
@@ -92,17 +92,29 @@ <h3 itemprop="headline">{newsItem.title}</h3>
 			</f:if>
 
 			<f:if condition="{settings.detail.showSocialShareButtons}">
-				<div class="facebook">
-					<div class="like">
-						<n:social.facebook.like />
-					</div>
-					<div class="share">
-						<n:social.facebook.share />
-					</div>
-					<div class="twitter">
-						<n:social.twitter>Twitter</n:social.twitter>
-					</div>
-				</div>
+				<f:comment>
+					Care about the privacy of your readers?
+					Checkout https://typo3.org/extensions/repository/view/rx_shariff
+					and it will be used automatically!
+				</f:comment>
+				<n:extensionLoaded extensionKey="rx_shariff">
+					<f:then>
+						<f:render partial="Detail/Shariff" />
+					</f:then>
+					<f:else>
+						<div class="facebook">
+							<div class="like">
+								<n:social.facebook.like />
+							</div>
+							<div class="share">
+								<n:social.facebook.share />
+							</div>
+							<div class="twitter">
+								<n:social.twitter>Twitter</n:social.twitter>
+							</div>
+						</div>
+					</f:else>
+				</n:extensionLoaded>
 			</f:if>
 
 			<f:if condition="{settings.detail.disqusShortname}">

diff --git a/Resources/Private/Extensions/News/TypoScript/setup.ts b/Resources/Private/Extensions/News/TypoScript/setup.ts
@@ -10,3 +10,7 @@ plugin.tx_news.settings.simpleList.cropMaxCharacters = 280
 plugin.tx_news.settings.timeline.cropMaxCharacters = 500
 
 plugin.tx_news.settings.list.media.dummyImage = typo3conf/ext/theme_t3kit/Resources/Public/Extensions/News/images/no_image.png
+
+#If extension rx_shariff is loaded, define services here
+#twitter,facebook,googleplus,linkedin,xing,pinterest,whatsapp,mail,addthis,tumblr,flattr,diaspora,reddit,stumbleupon,threema,info
+plugin.tx_news.settings.detail.shariffServices = facebook,twitter,whatsapp
diff --git a/Resources/Public/less/main.js b/Resources/Public/less/main.js
@@ -588,7 +588,7 @@ var mainSearchInputList = {}
       heightRatio: 0.6
     })
   })
-})(jQuery)
+})(jQuery);
 
 // ^^^^^^^^^^ general.js ^^^^^^^^^^^
 
diff --git a/Resources/Public/less/main.less b/Resources/Public/less/main.less
@@ -3352,7 +3352,7 @@
     padding: 4px 10px;
 }
 // news small fixes
-.news-single .twitter {
+.news-single .facebook > .twitter {
     padding-top: 6px;
 }
 // fix social links on news-single page

diff --git a/felayout_t3kit/dev/styles/main/plugins/news/news.less b/felayout_t3kit/dev/styles/main/plugins/news/news.less
@@ -430,7 +430,7 @@
     padding: 4px 10px;
 }
 // news small fixes
-.news-single .twitter {
+.news-single .facebook > .twitter {
     padding-top: 6px;
 }
 // fix social links on news-single page