UX: reviving old topic popup should not have red warning background

arpitjalan · arpitjalan · commit 3eb0288adb16 · 2015-11-25T16:31:17.000+05:30
diff --git a/app/assets/stylesheets/desktop/compose.scss b/app/assets/stylesheets/desktop/compose.scss
@@ -29,6 +29,10 @@
     background: dark-light-diff($danger, $secondary, 50%, -40%);
   }
 
+  &.old-topic {
+    background-color: dark-light-diff($tertiary, $secondary, 85%, -65%);
+  }
+
   h3 {
     margin-bottom: 10px;
   }
diff --git a/lib/composer_messages_finder.rb b/lib/composer_messages_finder.rb
@@ -149,7 +149,7 @@ def check_reviving_old_topic
     {
       templateName: 'composer/education',
       wait_for_typing: false,
-      extraClass: 'urgent',
+      extraClass: 'old-topic',
       body: PrettyText.cook(I18n.t('education.reviving_old_topic', days: (Time.zone.now - topic.last_posted_at).round / 1.day))
     }
   end

Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,10 @@`
`29`	`29`	`background: dark-light-diff($danger, $secondary, 50%, -40%);`
`30`	`30`	`}`
`31`	`31`
	`32`	`+ &.old-topic {`
	`33`	`+ background-color: dark-light-diff($tertiary, $secondary, 85%, -65%);`
	`34`	`+ }`
	`35`	`+`
`32`	`36`	`h3 {`
`33`	`37`	`margin-bottom: 10px;`
`34`	`38`	`}`
Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ def check_reviving_old_topic`
`149`	`149`	`{`
`150`	`150`	`templateName: 'composer/education',`
`151`	`151`	`wait_for_typing: false,`
`152`		`- extraClass: 'urgent',`
	`152`	`+ extraClass: 'old-topic',`
`153`	`153`	`body: PrettyText.cook(I18n.t('education.reviving_old_topic', days: (Time.zone.now - topic.last_posted_at).round / 1.day))`
`154`	`154`	`}`
`155`	`155`	`end`