From 841d5cfb7e1d0de36abf3cc795a5bd11c288baa6 Mon Sep 17 00:00:00 2001
From: kgough <kgough@redhat.com>
Date: Thu, 24 Aug 2017 16:20:16 +1000
Subject: [PATCH] feat(ZNTA-2171) css styling for Reject Translation History
 ActivityFeedItem story

---
 .../ActivityFeedItem.story.js                 | 32 +++++++++++++++++--
 .../components/ActivityFeedItem/index.js      | 24 +++++++++++---
 .../app/editor/containers/Sidebar/index.css   |  8 +++++
 3 files changed, 57 insertions(+), 7 deletions(-)

diff --git a/server/zanata-frontend/src/frontend/app/editor/components/ActivityFeedItem/ActivityFeedItem.story.js b/server/zanata-frontend/src/frontend/app/editor/components/ActivityFeedItem/ActivityFeedItem.story.js
index 183fed0bd70..665f8898e56 100644
--- a/server/zanata-frontend/src/frontend/app/editor/components/ActivityFeedItem/ActivityFeedItem.story.js
+++ b/server/zanata-frontend/src/frontend/app/editor/components/ActivityFeedItem/ActivityFeedItem.story.js
@@ -71,14 +71,40 @@ storiesOf('ActivityFeedItem', module)
       user={user} />
   ))
 
-  .add('rejected', () => (
+  .add('rejected - minor priority', () => (
     <ActivityFeedItem
-      criteria="Spellign and Grammah"
+      criteria="Spelling and Grammar"
       type="revision"
       priority="Minor"
-      textStatus="u-textWarning"
+      textStatus="u-textHighlight"
       status="rejected"
+      commentText="You spelt this wrong."
       content={'নাম'}
       lastModifiedTime={lastModifiedTime}
       user={user} />
   ))
+    .add('rejected - major priority', () => (
+        <ActivityFeedItem
+            criteria="Spelling and Grammar"
+            type="revision"
+            priority="Major"
+            textStatus="u-textWarning"
+            status="rejected"
+            commentText="You spelt this wrong."
+            content={'নাম'}
+            lastModifiedTime={lastModifiedTime}
+            user={user} />
+    ))
+    .add('rejected - critical priority', () => (
+        <ActivityFeedItem
+            criteria="Spelling and Grammar"
+            type="revision"
+            priority="Critical"
+            textStatus="u-textDanger"
+            status="rejected"
+            commentText="You spelt this wrong."
+            content={'নাম'}
+            lastModifiedTime={lastModifiedTime}
+            user={user} />
+    ))
+
diff --git a/server/zanata-frontend/src/frontend/app/editor/components/ActivityFeedItem/index.js b/server/zanata-frontend/src/frontend/app/editor/components/ActivityFeedItem/index.js
index 61a876bf36c..d4e754294b9 100644
--- a/server/zanata-frontend/src/frontend/app/editor/components/ActivityFeedItem/index.js
+++ b/server/zanata-frontend/src/frontend/app/editor/components/ActivityFeedItem/index.js
@@ -52,21 +52,37 @@ class ActivityFeedItem extends Component {
       imageUrl: PropTypes.string.isRequired,
     }).isRequired,
     criteria: PropTypes.string.isRequired,
+    commentText: PropTypes.string.isRequired,
     priority: PropTypes.oneOf(['Major', 'Critical', 'Minor']).isRequired,
 
   }
 
   getMessage = () => {
-    const { user, type, status, criteria } = this.props
+    const { user, type, status } = this.props
     // Uses href because editor app is separate from frontend app
     const name = (
       <Link useHref link={profileUrl(user.username)}>{user.name}</Link>
     )
     const priority = (
-        <span className={this.props.textStatus}>{this.props.priority}</span>
+        <span className="criteria-text">
+          <Icon name="warning" className="s0" />
+          <span className={this.props.textStatus}>
+            {this.props.priority}
+          </span>
+        </span>
     )
     const comment = (
-        <span className="comment"><Well>Comment</Well></span>
+        <span className="comment">
+          <Well bsSize="small">
+          <Icon name="comment" className="s0" />
+            {this.props.commentText}
+          </Well>
+        </span>
+    )
+    const criteria = (
+        <span className="criteria-text">
+          {this.props.criteria}
+        </span>
     )
     if (type === 'comment') {
       return (
@@ -120,7 +136,7 @@ class ActivityFeedItem extends Component {
               'Title for an item in the activity feed showing a reviewer ' +
               'rejected the translation. The inserted section is from ' +
               'ActivityFeedItem.rejected.rejectedTranslation'}
-            defaultMessage="{name} has {rejectedTranslation} because of {criteria} (Priority: {priority}). {comment}"
+            defaultMessage="{name} has {rejectedTranslation} for the reason: {criteria} - {priority} priority. {comment}"
             values={{
               name,
               rejectedTranslation,
diff --git a/server/zanata-frontend/src/frontend/app/editor/containers/Sidebar/index.css b/server/zanata-frontend/src/frontend/app/editor/containers/Sidebar/index.css
index 2f5a245c179..9ddf600b053 100644
--- a/server/zanata-frontend/src/frontend/app/editor/containers/Sidebar/index.css
+++ b/server/zanata-frontend/src/frontend/app/editor/containers/Sidebar/index.css
@@ -511,6 +511,14 @@ img.activity-avatar {
   border-left: solid 5px var(--Editor-color-approved);
 }
 
+.revision-box .well-sm {
+  background-color: rgba(255, 255, 255, 0.35) !important;
+}
+
+.criteria-text {
+  font-weight: 600;
+}
+
 .trans-comment-box {
   padding: 0.75rem;
   margin-top: 1rem;