From 6f779f7c315569eeff5a06588ee01ea0318720d9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois=20Schmidts?= <francois.schmidts@gmail.com>
Date: Wed, 10 Jun 2020 23:20:33 +0200
Subject: [PATCH] don't show article title in tabs, it's ugly

---
 .../clusterlist/components/Article.js         |  5 ++++-
 .../clusterlist/components/Articles.js        | 22 ++++++++-----------
 .../clusterlist/components/Content.js         |  1 +
 3 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/jsclient/src/features/clusterlist/components/Article.js b/jsclient/src/features/clusterlist/components/Article.js
index 45da83378..6ef01250e 100644
--- a/jsclient/src/features/clusterlist/components/Article.js
+++ b/jsclient/src/features/clusterlist/components/Article.js
@@ -6,7 +6,7 @@ import Divider from "@material-ui/core/Divider";
 
 import makeStyles from "./style";
 
-function Article({ article, hidden }) {
+function Article({ article, hidden, showTitle }) {
   const classes = makeStyles();
   let comments;
   if (article.comments) {
@@ -18,6 +18,7 @@ function Article({ article, hidden }) {
   }
   return (
     <div hidden={hidden} className={classes.article}>
+      <h1 hidden={!showTitle}>{article.title}</h1>
       <p>
         <span>Link</span>
         <Link color="secondary" target="_blank"
@@ -38,10 +39,12 @@ function Article({ article, hidden }) {
 
 Article.propTypes = {
   article: PropTypes.shape({
+    title: PropTypes.string.isRequired,
     link: PropTypes.string.isRequired,
     content: PropTypes.string.isRequired,
     comments: PropTypes.string,
   }),
+  showTitle: PropTypes.bool.isRequired,
   hidden: PropTypes.bool.isRequired,
 };
 
diff --git a/jsclient/src/features/clusterlist/components/Articles.js b/jsclient/src/features/clusterlist/components/Articles.js
index 5efad3633..ad1bfd4bd 100644
--- a/jsclient/src/features/clusterlist/components/Articles.js
+++ b/jsclient/src/features/clusterlist/components/Articles.js
@@ -25,21 +25,18 @@ function Articles({ articles, icons, content }) {
 
   let contentTitle, contentTab;
   if (articles.length === 1 && !content) {
-    return <Article article={articles[0]} hidden={false} />;
+    return <Article article={articles[0]} />;
   } else if (content) {
     contentTitle = (
-      <Tab key="t-0"
+      <Tab key="t-0" label=""
         className={classes.tabs}
-      //
         icon={<img src={jarrIcon} alt={proccessedContentTitle}
                    title={proccessedContentTitle} />}
-        label={proccessedContentTitle}
         value={0} aria-controls="a-0"
       />
     );
     contentTab = <Content content={content} hidden={0 !== currentIndex} />;
   }
-  const isOnlyOneTitle = [...new Set(articles.map((a) => a.title))].length === 1;
   return (
     <>
       <Tabs indicatorColor="primary" textColor="primary"
@@ -48,20 +45,19 @@ function Articles({ articles, icons, content }) {
         {contentTitle}
         {articles.map((article, index) => {
           index = content ? index + 1 : index;
-          return (<Tab key={`t-${index}`}
-            className={classes.tabs}
-            icon={<ClusterIcon iconUrl={icons[article["feed_id"]]} />}
-            label={isOnlyOneTitle ? null : article.title}
-            value={index}
-            aria-controls={`a-${index}`}
-          />);
+          return (<Tab key={`t-${index}`} label=""
+                       className={classes.tabs}
+                       icon={<ClusterIcon iconUrl={icons[article["feed_id"]]} />}
+                       value={index}
+                       aria-controls={`a-${index}`}
+                  />);
         })}
       </Tabs>
       {contentTab}
       {articles.map((article, index) => {
         index = content ? index + 1 : index;
         return (
-          <Article
+          <Article showTitle
             key={`a-${index}-${index !== currentIndex ? "h" : ""}`}
             id={`a-${index}`}
             article={article}
diff --git a/jsclient/src/features/clusterlist/components/Content.js b/jsclient/src/features/clusterlist/components/Content.js
index d44c4e1cc..f379e6033 100644
--- a/jsclient/src/features/clusterlist/components/Content.js
+++ b/jsclient/src/features/clusterlist/components/Content.js
@@ -64,6 +64,7 @@ function Content({ content, hidden }) {
   }
   return (
     <div hidden={hidden} className={classes.article}>
+      <h1>Proccessed Content</h1>
       {head}
       {comments}
       <Divider />