Esanders has uploaded a new change for review.

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

Change subject: Remove unused CSS classes
......................................................................

Remove unused CSS classes

By searching for CSS classes that don't exist in the code.

In some cases the classes were used but had been renamed.

Change-Id: I9f6823bfb6efcee0e5f479e23afbe91f1581ac57
---
M src/Frame.js
M src/layouts/FieldLayout.js
M src/styles/Frame.less
M src/styles/Toolbar.less
M src/styles/layouts/FieldsetLayout.less
5 files changed, 9 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/oojs/ui refs/changes/92/150192/1

diff --git a/src/Frame.js b/src/Frame.js
index 0b9aecb..83a3854 100644
--- a/src/Frame.js
+++ b/src/Frame.js
@@ -196,6 +196,9 @@
 
        // Initialize contents
        doc.open();
+       // The following classes can be used here:
+       // oo-ui-ltr
+       // oo-ui-rtl
        doc.write(
                '<!doctype html>' +
                '<html>' +
diff --git a/src/layouts/FieldLayout.js b/src/layouts/FieldLayout.js
index b2405c1..bc79c77 100644
--- a/src/layouts/FieldLayout.js
+++ b/src/layouts/FieldLayout.js
@@ -125,6 +125,11 @@
                        this.$element.removeClass( 'oo-ui-fieldLayout-align-' + 
this.align );
                }
                this.align = value;
+               // The following classes can be used here:
+               // oo-ui-fieldLayout-align-left
+               // oo-ui-fieldLayout-align-right
+               // oo-ui-fieldLayout-align-top
+               // oo-ui-fieldLayout-align-inline
                this.$element.addClass( 'oo-ui-fieldLayout-align-' + this.align 
);
        }
 
diff --git a/src/styles/Frame.less b/src/styles/Frame.less
index 8c49648..dc0525c 100644
--- a/src/styles/Frame.less
+++ b/src/styles/Frame.less
@@ -2,10 +2,9 @@
        margin: 0;
        padding: 0;
 
-       &-body {
+       &-content {
                margin: 0;
                padding: 0;
-               background: none;
        }
 
        /* Content div takes focus when opened, so hide outline  */
diff --git a/src/styles/Toolbar.less b/src/styles/Toolbar.less
index 236cbf0..0ace0ac 100644
--- a/src/styles/Toolbar.less
+++ b/src/styles/Toolbar.less
@@ -7,10 +7,6 @@
                line-height: 1em;
        }
 
-       &-bottom .oo-ui-toolbar-bar {
-               position: absolute;
-       }
-
        &-actions {
                float: right;
        }
diff --git a/src/styles/layouts/FieldsetLayout.less 
b/src/styles/layouts/FieldsetLayout.less
index d35a036..9bd786e 100644
--- a/src/styles/layouts/FieldsetLayout.less
+++ b/src/styles/layouts/FieldsetLayout.less
@@ -7,10 +7,6 @@
                margin-top: 2em;
        }
 
-       &-labeled {
-               margin-top: -0.75em;
-       }
-
        > .oo-ui-labeledElement-label {
                margin-bottom: 0.5em;
                padding: 0.25em 0;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9f6823bfb6efcee0e5f479e23afbe91f1581ac57
Gerrit-PatchSet: 1
Gerrit-Project: oojs/ui
Gerrit-Branch: master
Gerrit-Owner: Esanders <esand...@wikimedia.org>

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

Reply via email to