From b2e05a8a6dfee81a589be91774477d31d121403a Mon Sep 17 00:00:00 2001
From: Benjamin Ludewig <benjamin.ludewig@neuland-bfi.de>
Date: Wed, 23 Jan 2019 15:28:34 +0100
Subject: [PATCH] feature: Show tooltips and featuretoggle names in event view

---
 .../component/history/history-list-component.jsx   | 14 ++++++++++++--
 .../history/history-list-toggle-component.jsx      |  2 +-
 2 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/frontend/src/component/history/history-list-component.jsx b/frontend/src/component/history/history-list-component.jsx
index 20d4539cfd..15e7c4d7a2 100644
--- a/frontend/src/component/history/history-list-component.jsx
+++ b/frontend/src/component/history/history-list-component.jsx
@@ -15,6 +15,7 @@ class HistoryList extends Component {
         settings: PropTypes.object,
         location: PropTypes.object,
         updateSetting: PropTypes.func.isRequired,
+        hideName: PropTypes.bool,
     };
 
     toggleShowDiff() {
@@ -25,7 +26,7 @@ class HistoryList extends Component {
     }
     render() {
         const showData = this.props.settings.showData;
-        const { history } = this.props;
+        const { history, hideName } = this.props;
         if (!history || history.length < 0) {
             return null;
         }
@@ -34,6 +35,7 @@ class HistoryList extends Component {
             <span
                 className={commonStyles.truncate}
                 style={{ display: 'inline-block', verticalAlign: 'middle', width: '100%' }}
+                title={v}
             >
                 {v}
             </span>
@@ -51,6 +53,7 @@ class HistoryList extends Component {
                         Object.assign(
                             {
                                 diff: <HistoryItemDiff entry={entry} />,
+                                name: entry.data.name,
                             },
                             entry
                         )
@@ -58,12 +61,19 @@ class HistoryList extends Component {
                     className={commonStyles.fullwidth}
                     style={{ border: 0, tableLayout: 'fixed', minWidth: '840px' }}
                 >
-                    <TableHeader name="type" cellFormatter={truncateTableCell} style={{ width: '136px' }}>
+                    <TableHeader name="type" cellFormatter={truncateTableCell} style={{ width: '115px' }}>
                         Type
                     </TableHeader>
                     <TableHeader name="createdBy" cellFormatter={truncateTableCell} style={{ width: '115px' }}>
                         User
                     </TableHeader>
+                    {hideName ? (
+                        []
+                    ) : (
+                        <TableHeader name="name" cellFormatter={truncateTableCell} style={{ width: '153px' }}>
+                            Feature Toggle
+                        </TableHeader>
+                    )}
                     <TableHeader name="diff">Diff</TableHeader>
                     <TableHeader
                         numeric
diff --git a/frontend/src/component/history/history-list-toggle-component.jsx b/frontend/src/component/history/history-list-toggle-component.jsx
index ee4a69b99a..ec8e7d6684 100644
--- a/frontend/src/component/history/history-list-toggle-component.jsx
+++ b/frontend/src/component/history/history-list-toggle-component.jsx
@@ -18,7 +18,7 @@ class HistoryListToggle extends Component {
             return <span>fetching..</span>;
         }
         const { history } = this.props;
-        return <HistoryList history={history} title="Change log" />;
+        return <HistoryList history={history} hideName title="Change log" />;
     }
 }