[MediaWiki-commits] [Gerrit] Styling adjustments for notifications - change (mediawiki...Echo)

2016-03-04 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged.

Change subject: Styling adjustments for notifications
..


Styling adjustments for notifications

Bug: T125969
Bug: T128444
Change-Id: I3dba4fcddf5262450cce4ba384abfdc3518b7cb8
---
M modules/ooui/styles/mw.echo.ui.MenuItemWidget.less
M modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less
M modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less
3 files changed, 13 insertions(+), 11 deletions(-)

Approvals:
  Sbisson: Looks good to me, approved
  Catrope: Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/modules/ooui/styles/mw.echo.ui.MenuItemWidget.less 
b/modules/ooui/styles/mw.echo.ui.MenuItemWidget.less
index dbe31c3..52457d7 100644
--- a/modules/ooui/styles/mw.echo.ui.MenuItemWidget.less
+++ b/modules/ooui/styles/mw.echo.ui.MenuItemWidget.less
@@ -30,7 +30,9 @@
}
 
&-prioritized {
+   .mw-echo-ui-mixin-hover-opacity();
display: inline-block;
+   padding: 0;
}
 
// Correct for when inside the popup menu
diff --git a/modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less 
b/modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less
index 71d65c1..b677da8 100644
--- a/modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less
+++ b/modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less
@@ -13,7 +13,7 @@
}
 
&-content {
-   margin-bottom: 0.5em;
+   margin-bottom: 1em;
padding-right: 0.8em;
padding-left: ~"calc(30px + 1.6em)"; // 30px+0.8em from 
ItemWidget, plus 0.8em
}
diff --git a/modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less 
b/modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less
index 1a9a070..d9be2c9 100644
--- a/modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less
+++ b/modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less
@@ -6,7 +6,6 @@
background-color: #F1F1F1;
border-bottom: 1px solid #DD;
white-space: normal;
-   line-height: 16px;
 
&:hover > a {
text-decoration: none;
@@ -33,6 +32,7 @@
width: 100%;
 
&-message {
+   line-height: 16px;
&-header {
color: @notification-text-color;
}
@@ -55,20 +55,18 @@
 
& > &-buttons.oo-ui-buttonSelectWidget {
display: table-cell;
-   vertical-align: top;
-   width: 100%;
+   vertical-align: middle;
}
 
.mw-echo-ui-menuItemWidget {
-   &:not(:last-child) {
-   margin-right: 1em;
-   }
+   margin-right: 1.2em;
}
 
&-menu {
+   .mw-echo-ui-mixin-hover-opacity();
display: table-cell;
-   vertical-align: top;
-   padding: 0 0.5em;
+   vertical-align: middle;
+   padding: 0;
 
.oo-ui-popupWidget-popup {
font-size: 1 / 0.8em;
@@ -78,10 +76,12 @@
 
&-timestamp {
display: table-cell;
-   vertical-align: top;
+   vertical-align: middle;
+   text-align: right;
color: black;
opacity: @opacity-low;
white-space: nowrap;
+   width: 100%;
}
 
}
@@ -155,7 +155,7 @@
float: right;
font-size: 1em;
margin-top: -0.5em;
-   margin-right: -0.5em;
+   margin-right: -0.4em;
padding: 0;
 
.mw-echo-ui-notificationItemWidget-bundle & {

-- 
To view, visit https://gerrit.wikimedia.org/r/274804
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3dba4fcddf5262450cce4ba384abfdc3518b7cb8
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Mooeypoo 
Gerrit-Reviewer: Catrope 
Gerrit-Reviewer: Sbisson 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/l

[MediaWiki-commits] [Gerrit] Styling adjustments for notifications - change (mediawiki...Echo)

2016-03-03 Thread Mooeypoo (Code Review)
Mooeypoo has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/274804

Change subject: Styling adjustments for notifications
..

Styling adjustments for notifications

Bug: T125969
Change-Id: I3dba4fcddf5262450cce4ba384abfdc3518b7cb8
---
M modules/ooui/styles/mw.echo.ui.MenuItemWidget.less
M modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less
M modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less
3 files changed, 13 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo 
refs/changes/04/274804/1

diff --git a/modules/ooui/styles/mw.echo.ui.MenuItemWidget.less 
b/modules/ooui/styles/mw.echo.ui.MenuItemWidget.less
index dbe31c3..52457d7 100644
--- a/modules/ooui/styles/mw.echo.ui.MenuItemWidget.less
+++ b/modules/ooui/styles/mw.echo.ui.MenuItemWidget.less
@@ -30,7 +30,9 @@
}
 
&-prioritized {
+   .mw-echo-ui-mixin-hover-opacity();
display: inline-block;
+   padding: 0;
}
 
// Correct for when inside the popup menu
diff --git a/modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less 
b/modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less
index 71d65c1..b677da8 100644
--- a/modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less
+++ b/modules/ooui/styles/mw.echo.ui.NotificationGroupItemWidget.less
@@ -13,7 +13,7 @@
}
 
&-content {
-   margin-bottom: 0.5em;
+   margin-bottom: 1em;
padding-right: 0.8em;
padding-left: ~"calc(30px + 1.6em)"; // 30px+0.8em from 
ItemWidget, plus 0.8em
}
diff --git a/modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less 
b/modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less
index 1a9a070..d9be2c9 100644
--- a/modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less
+++ b/modules/ooui/styles/mw.echo.ui.NotificationItemWidget.less
@@ -6,7 +6,6 @@
background-color: #F1F1F1;
border-bottom: 1px solid #DD;
white-space: normal;
-   line-height: 16px;
 
&:hover > a {
text-decoration: none;
@@ -33,6 +32,7 @@
width: 100%;
 
&-message {
+   line-height: 16px;
&-header {
color: @notification-text-color;
}
@@ -55,20 +55,18 @@
 
& > &-buttons.oo-ui-buttonSelectWidget {
display: table-cell;
-   vertical-align: top;
-   width: 100%;
+   vertical-align: middle;
}
 
.mw-echo-ui-menuItemWidget {
-   &:not(:last-child) {
-   margin-right: 1em;
-   }
+   margin-right: 1.2em;
}
 
&-menu {
+   .mw-echo-ui-mixin-hover-opacity();
display: table-cell;
-   vertical-align: top;
-   padding: 0 0.5em;
+   vertical-align: middle;
+   padding: 0;
 
.oo-ui-popupWidget-popup {
font-size: 1 / 0.8em;
@@ -78,10 +76,12 @@
 
&-timestamp {
display: table-cell;
-   vertical-align: top;
+   vertical-align: middle;
+   text-align: right;
color: black;
opacity: @opacity-low;
white-space: nowrap;
+   width: 100%;
}
 
}
@@ -155,7 +155,7 @@
float: right;
font-size: 1em;
margin-top: -0.5em;
-   margin-right: -0.5em;
+   margin-right: -0.4em;
padding: 0;
 
.mw-echo-ui-notificationItemWidget-bundle & {

-- 
To view, visit https://gerrit.wikimedia.org/r/274804
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3dba4fcddf5262450cce4ba384abfdc3518b7cb8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Mooeypoo 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits