Santhosh has uploaded a new change for review.

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

Change subject: MT Card: Avoid having a dedicated element for clearfix
......................................................................

MT Card: Avoid having a dedicated element for clearfix

All tools cards have clearfix now.

Change-Id: I9636e6af82901ad2ef2fce37c510701fb684d44c
---
M modules/tools/ext.cx.tools.mt.js
M modules/tools/styles/ext.cx.tools.card.less
M modules/tools/styles/ext.cx.tools.mt.less
3 files changed, 10 insertions(+), 15 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/ContentTranslation 
refs/changes/38/259638/1

diff --git a/modules/tools/ext.cx.tools.mt.js b/modules/tools/ext.cx.tools.mt.js
index e71eecb..1d7164e 100644
--- a/modules/tools/ext.cx.tools.mt.js
+++ b/modules/tools/ext.cx.tools.mt.js
@@ -277,7 +277,7 @@
        }
 
        MTControlCard.prototype.getCard = function () {
-               var $titleRow, $title, $controlButtonsBlock, $bottom;
+               var $titleRow, $title, $controlButtonsBlock;
 
                this.$card = $( '<div>' )
                        .addClass( 'card mt' );
@@ -313,14 +313,10 @@
                                this.actions.$clear
                        );
 
-               $bottom = $( '<div>' )
-                       .addClass( 'card__bottom' );
-
                this.$card.append(
                        $titleRow,
                        this.$providerSelectorTrigger,
-                       $controlButtonsBlock,
-                       $bottom
+                       $controlButtonsBlock
                );
 
                this.buildProvidersMenu();
diff --git a/modules/tools/styles/ext.cx.tools.card.less 
b/modules/tools/styles/ext.cx.tools.card.less
index 2281b85..158b94c 100644
--- a/modules/tools/styles/ext.cx.tools.card.less
+++ b/modules/tools/styles/ext.cx.tools.card.less
@@ -11,6 +11,14 @@
                border-color: #BBBBBB;
                box-shadow: 0 1px 0 #BBBBBB;
        }
+       /* Clearfix */
+       &:after {
+               content: " "; /* Older browser do not support empty content */
+               visibility: hidden;
+               display: block;
+               height: 0;
+               clear: both;
+       }
 }
 
 .card__title,
diff --git a/modules/tools/styles/ext.cx.tools.mt.less 
b/modules/tools/styles/ext.cx.tools.mt.less
index 72af107..b174a08 100644
--- a/modules/tools/styles/ext.cx.tools.mt.less
+++ b/modules/tools/styles/ext.cx.tools.mt.less
@@ -130,13 +130,4 @@
                        border: none;
                }
        }
-
-       .card__bottom {
-               .mw-ui-item;
-               .mw-ui-one-whole;
-               float: none; // XXX why would it be needed?
-               clear: both;
-               padding: 5px;
-               text-align: center;
-       }
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9636e6af82901ad2ef2fce37c510701fb684d44c
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ContentTranslation
Gerrit-Branch: master
Gerrit-Owner: Santhosh <[email protected]>

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

Reply via email to