MaxSem has submitted this change and it was merged.

Change subject: Fixing Thank button styling
......................................................................


Fixing Thank button styling

This also requires removing the scoping from the default button
styles, which seems to not adversely affect anything.

Bug: 55596
Change-Id: I4f2431717868b2a9c257e5804b5ab1a01db01f8e
---
M less/common/buttons.less
M less/specials/mobilediff.less
2 files changed, 54 insertions(+), 74 deletions(-)

Approvals:
  MaxSem: Verified; Looks good to me, approved
  jenkins-bot: Verified



diff --git a/less/common/buttons.less b/less/common/buttons.less
index 74eaf36..cae4979 100644
--- a/less/common/buttons.less
+++ b/less/common/buttons.less
@@ -10,54 +10,47 @@
        line-height: 1;
        cursor: pointer;
        margin: 0;
-}
+       padding: .8em 1.2em;
+       text-align: center;
+       border: 1px #33589f solid;
+       color: white;
+       .vertical-gradient( #3670C8, @blueBase );
 
-.content, .buttonBar {
-       .button,
-       button,
-       input[type=submit] {
-               padding: .8em 1.2em;
-               text-align: center;
-               border: 1px #33589f solid;
-               .text-with-shadow( white );
-               .vertical-gradient( #3670C8, @blueBase );
+       &:hover {
+               .vertical-gradient( #4c84da, #3f77d7 );
+       }
+
+       &:active {
+               .vertical-gradient( #2d5ea9, #2a549c );
+       }
+
+       &:visited {
+               color: #fff;
+       }
+
+       &:disabled {
+               @disabledColor: lighten( desaturate( #3f77d7, 15% ), 10% );
+               color: lighten( #3f77d7, 30% );;
+               .vertical-gradient( @disabledColor,@disabledColor );
+               border-color: darken( @disabledColor, 5% );
+       }
+
+       &.cancel {
+               background: #777;
+               border: 1px #666 solid;
+       }
+
+       &.inline {
+               display: inline;
+               padding: 0;
+               line-height: inherit;
+               color: #002bb8;
+               background: none;
+               border: none;
+               text-shadow: none;
 
                &:hover {
-                       .vertical-gradient( #4c84da, #3f77d7 );
-               }
-
-               &:active {
-                       .vertical-gradient( #2d5ea9, #2a549c );
-               }
-
-               &:visited {
-                       color: #fff;
-               }
-
-               &:disabled {
-                       @disabledColor: lighten( desaturate( #3f77d7, 15% ), 
10% );
-                       color: lighten( #3f77d7, 30% );;
-                       .vertical-gradient( @disabledColor,@disabledColor );
-                       border-color: darken( @disabledColor, 5% );
-               }
-
-               &.cancel {
-                       background: #777;
-                       border: 1px #666 solid;
-               }
-
-               &.inline {
-                       display: inline;
-                       padding: 0;
-                       line-height: inherit;
-                       color: #002bb8;
-                       background: none;
-                       border: none;
-                       text-shadow: none;
-
-                       &:hover {
-                               text-decoration: underline;
-                       }
+                       text-decoration: underline;
                }
        }
 }
diff --git a/less/specials/mobilediff.less b/less/specials/mobilediff.less
index ac7b728..de40d31 100644
--- a/less/specials/mobilediff.less
+++ b/less/specials/mobilediff.less
@@ -183,39 +183,26 @@
        }
 }
 
-/* Need .content to match the button selectors in ui.css */
-.content {
+#mw-mf-diffview {
        .mw-mf-action-button {
                float: right;
                border: 1px #25A260 solid;
                background-color: #27AA65;
-               background-image: -moz-linear-gradient(top, #2EC977 0, #27AA65 
100%);
-               background-image: -webkit-gradient(linear, left top, left 
bottom, color-stop(0, #2EC977), color-stop(100%, #27AA65));
-               background-image: -webkit-linear-gradient(top, #2EC977 0, 
#27AA65 100%);
-               background-image: linear-gradient(#2EC977 0, #27AA65 100%);
-       }
-       .mw-mf-action-button:hover {
-               background-color: #2EC977;
-               background-image: -moz-linear-gradient(top, #3ED384 0, #2EC977 
100%);
-               background-image: -webkit-gradient(linear, left top, left 
bottom, color-stop(0, #3ED384), color-stop(100%, #2EC977));
-               background-image: -webkit-linear-gradient(top, #3ED384 0, 
#2EC977 100%);
-               background-image: linear-gradient(#3ED384 0, #2EC977 100%);
-       }
-       .mw-mf-action-button:active {
-               background-color: #249E5E;
-               background-image: -moz-linear-gradient(top, #31B570 0, #249E5E 
100%);
-               background-image: -webkit-gradient(linear, left top, left 
bottom, color-stop(0, #31B570), color-stop(100%, #249E5E));
-               background-image: -webkit-linear-gradient(top, #31B570 0, 
#249E5E 100%);
-               background-image: linear-gradient(#31B570 0, #249E5E 100%);
-       }
-       .mw-mf-action-button:disabled {
-               color: white;
-               border: 1px #80D1A7 solid;
-               background-color: #80D1A7;
-               background-image: -moz-linear-gradient(top, #90DEB5 0, #80D1A7 
100%);
-               background-image: -webkit-gradient(linear, left top, left 
bottom, color-stop(0, #90DEB5), color-stop(100%, #80D1A7));
-               background-image: -webkit-linear-gradient(top, #90DEB5 0, 
#80D1A7 100%);
-               background-image: linear-gradient(#90DEB5 0, #80D1A7 100%);
+               .vertical-gradient( #2EC977, #27AA65 );
+               &:hover {
+                       background-color: #2EC977;
+                       .vertical-gradient( #3ED384, #2EC977 );
+               }
+               &:active {
+                       background-color: #249E5E;
+                       .vertical-gradient( #31B570, #249E5E );
+               }
+               &:disabled {
+                       color: white;
+                       border: 1px #80D1A7 solid;
+                       background-color: #80D1A7;
+                       .vertical-gradient( #90DEB5, #80D1A7 );
+               }
        }
        .mw-mf-action-button-icon {
                display: inline-block;

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4f2431717868b2a9c257e5804b5ab1a01db01f8e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: 1.22wmf20
Gerrit-Owner: MaxSem <[email protected]>
Gerrit-Reviewer: Kaldari <[email protected]>
Gerrit-Reviewer: MaxSem <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to