VolkerE has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/402756 )

Change subject: WikimediaUI theme: Align size and position of overhauled icons
......................................................................

WikimediaUI theme: Align size and position of overhauled icons

Bug: T177432
Change-Id: I154f638abd1f552018a3ef9d046627b88cfe9a13
---
M src/themes/wikimediaui/common.less
M src/themes/wikimediaui/elements.less
M src/themes/wikimediaui/widgets.less
3 files changed, 8 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/oojs/ui refs/changes/56/402756/1

diff --git a/src/themes/wikimediaui/common.less 
b/src/themes/wikimediaui/common.less
index a904fec..c69de80 100644
--- a/src/themes/wikimediaui/common.less
+++ b/src/themes/wikimediaui/common.less
@@ -64,14 +64,14 @@
 @size-fieldlayout-help-icon: 22 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base;
 @size-tool: 40 / @oo-ui-font-size-browser / @oo-ui-font-size-base; // equals 
`3.125em`≈`40px`
 
-@start-framed-icon-only: 7 / @oo-ui-font-size-browser / @oo-ui-font-size-base; 
// equals `0.547em`≈`7px`; HACK: We have to break with 
`@padding-start-input-text-icon` to perfectly center in `@min-width-button-base`
+@start-framed-icon-only: 10 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base; // equals `0.547em`≈`7px`; HACK: We have to break with 
`@padding-start-input-text-icon` to perfectly center in `@min-width-button-base`
 @start-framed-icon-only-numberinput: 6 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base; // equals `0.46875em`≈`6px`
 @start-framed-indicator-only: 13 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base; // equals `1.09375em`≈`14px`; `@size-indicator + 2 * 
@border-width-base`
 @start-framed-square-indicator-only: 9 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base; // equals `0.703125em`≈`9px`
 @start-frameless: -@start-frameless-icon;
 @start-frameless-icon: 3 / @oo-ui-font-size-browser / @oo-ui-font-size-base;
 @start-tagmultiselect-input: 5 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base; // equals `0.390625em`≈`5px`; 
`@padding-horizontal-tagitem` + `@border-width-base`
-@start-tool-icon-indicator: 8 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base;
+@start-tool-icon-indicator: 12 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base;
 @start-toolbar-narrow-tool-icon: 4 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base;
 @end-tagitem: 4 / @oo-ui-font-size-browser / @oo-ui-font-size-base; // equals 
`0.3125em`≈`4px`
 
@@ -133,7 +133,6 @@
 @padding-bottom-base: 7 / @oo-ui-font-size-browser / @oo-ui-font-size-base; // 
equals `0.547em`≈`7px`
 @padding-bottom-menu: 8 / @oo-ui-font-size-browser / @oo-ui-font-size-base;
 @padding-bottom-tool-label: 12 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base;
-@padding-start-input-text-icon: 6 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base; // As long as we have whitespace in icon canvas, we need 
reduced `padding-left`.
 @padding-start-icon: @size-icon;
 @padding-start-icon-only: 26 / @oo-ui-font-size-browser / 
@oo-ui-font-size-base; // equals `2.03125em`≈`26px`; @min-width-button-base - 
@padding-horizontal-base - 2 * @border-width-base;
 @padding-start-fieldset-header-iconized: 26 / @oo-ui-font-size-browser / ( 
@oo-ui-font-size-base * @font-size-fieldset-header );
diff --git a/src/themes/wikimediaui/elements.less 
b/src/themes/wikimediaui/elements.less
index 585adef..946b7dc 100644
--- a/src/themes/wikimediaui/elements.less
+++ b/src/themes/wikimediaui/elements.less
@@ -271,7 +271,7 @@
                                        padding-left: 
@padding-start-input-text-icon-label;
 
                                        > .oo-ui-iconElement-icon {
-                                               left: 
@padding-start-input-text-icon;
+                                               //left: 
@padding-horizontal-input-text;
                                        }
                                }
                        }
diff --git a/src/themes/wikimediaui/widgets.less 
b/src/themes/wikimediaui/widgets.less
index 7348fbb..810cd2b 100644
--- a/src/themes/wikimediaui/widgets.less
+++ b/src/themes/wikimediaui/widgets.less
@@ -265,7 +265,7 @@
                padding-left: @padding-start-input-text-icon-label;
 
                > .oo-ui-iconElement-icon {
-                       left: @padding-start-input-text-icon;
+                       left: @padding-horizontal-input-text;
                }
        }
 
@@ -570,7 +570,7 @@
                line-height: @line-height-reset;
 
                .oo-ui-iconElement-icon {
-                       left: @padding-start-input-text-icon;
+                       left: @padding-horizontal-input-text;
                }
 
                .oo-ui-indicatorElement-indicator {
@@ -1364,7 +1364,7 @@
                > .oo-ui-iconElement-icon {
                        display: none;
                        top: -@border-width-base; // Address `border-top` of 
parent element
-                       left: @padding-start-input-text-icon;
+                       left: @padding-horizontal-input-text;
                        min-height: @min-height-widget-base;
                        margin-left: -@border-width-base; // Address extra 
`border-left` pixel above
                }
@@ -1397,7 +1397,7 @@
 
        &-clearButton {
                top: -1px; // Address `border-width`
-               right: @padding-start-input-text-icon; // With close, no 
indicator
+               right: @padding-horizontal-input-text; // With close, no 
indicator
                min-width: @min-size-icon;
                width: @size-icon;
                margin-right: 0;
@@ -1907,7 +1907,7 @@
                }
 
                > .oo-ui-iconElement-icon {
-                       left: @padding-start-input-text-icon;
+                       left: @padding-horizontal-input-text;
                }
 
                textarea + .oo-ui-iconElement-icon {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I154f638abd1f552018a3ef9d046627b88cfe9a13
Gerrit-PatchSet: 1
Gerrit-Project: oojs/ui
Gerrit-Branch: master
Gerrit-Owner: VolkerE <volke...@wikimedia.org>

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

Reply via email to