From ad776348542d4841da6d4706cdb4e91faacec902 Mon Sep 17 00:00:00 2001 From: Robin Ward Date: Fri, 15 Sep 2017 13:24:43 -0400 Subject: [PATCH] FIX: More CSS regressions with notifications in non-english --- .../widgets/user-notifications-large.js.es6 | 2 +- app/assets/stylesheets/common/base/user.scss | 29 +++++++++++++++++++ 2 files changed, 30 insertions(+), 1 deletion(-) diff --git a/app/assets/javascripts/discourse/widgets/user-notifications-large.js.es6 b/app/assets/javascripts/discourse/widgets/user-notifications-large.js.es6 index d10e64b5818..dd90f281591 100644 --- a/app/assets/javascripts/discourse/widgets/user-notifications-large.js.es6 +++ b/app/assets/javascripts/discourse/widgets/user-notifications-large.js.es6 @@ -4,7 +4,7 @@ import { dateNode } from 'discourse/helpers/node'; createWidget('large-notification-item', { buildClasses(attrs) { - const result = ['item', 'notification']; + const result = ['item', 'notification', 'large-notification']; if (!attrs.get('read')) { result.push('unread'); } diff --git a/app/assets/stylesheets/common/base/user.scss b/app/assets/stylesheets/common/base/user.scss index 134771b80c3..bf2a27c58fa 100644 --- a/app/assets/stylesheets/common/base/user.scss +++ b/app/assets/stylesheets/common/base/user.scss @@ -504,3 +504,32 @@ margin-top: 0; } } + +.large-notification { + display: flex; + align-items: center; + + a { + display: flex; + align-items: center; + + .d-icon { + margin-right: 0.5em; + } + + span { + color: $primary; + } + + // Can remove this once other languages have removed html from i18n values + div { + .fa { + display: none; + } + p { + margin: 0; + } + } + } +} +