TablePager: Reduce pagination button in visual strength
authorVolker E <volker.e@wikimedia.org>
Fri, 18 Jan 2019 00:04:30 +0000 (16:04 -0800)
committerVolker E <volker.e@wikimedia.org>
Fri, 18 Jan 2019 00:56:21 +0000 (16:56 -0800)
Reducing buttons to `frameless` buttons and reposition icons of
next and last page button to be symmetrically harmonious.

Bug: T214128
Change-Id: I1c0a621d7699b5fadc2559abdf6003a0db0aabae

includes/pager/TablePager.php
resources/src/mediawiki.pager.tablePager/TablePager.less

index e99c33b..8934fc2 100644 (file)
@@ -310,6 +310,9 @@ abstract class TablePager extends IndexPager {
                                // * table_pager_prev
                                // * table_pager_next
                                // * table_pager_last
+                               'classes' => [ 'TablePager-button-' . $type ],
+                               'flags' => [ 'progressive' ],
+                               'framed' => false,
                                'label' => $this->msg( 'table_pager_' . $type )->text(),
                                'href' => $queries[ $type ] ?
                                        $title->getLinkURL( $queries[ $type ] + $this->getDefaultQuery() ) :
index 5b3519e..bea197a 100644 (file)
        .background-image-svg('images/arrow-sort-descending.svg', 'images/arrow-sort-descending.png');
 }
 
-.TablePager_nav.oo-ui-buttonGroupWidget {
-       display: block;
-       text-align: center;
-       margin: 1em;
+.TablePager_nav {
+       &.oo-ui-buttonGroupWidget {
+               display: block;
+               margin: 1em;
+               text-align: center;
+       }
+
+       // Overrule default OOUI frameless special treatment for left-aligned widgets
+       & .oo-ui-buttonElement-frameless {
+               &.oo-ui-labelElement > .oo-ui-buttonElement-button {
+                       padding-right: 0.5em;
+               }
+
+               &.oo-ui-labelElement.oo-ui-iconElement:first-child {
+                       margin-left: 0;
+               }
+       }
+
+       .TablePager-button-next,
+       .TablePager-button-last {
+               &.oo-ui-buttonElement-frameless.oo-ui-iconElement > .oo-ui-buttonElement-button {
+                       padding-left: 0.5em;
+                       padding-right: 2.14285714em;
+
+                       > .oo-ui-iconElement-icon {
+                               left: auto;
+                               right: 0.35714286em;
+                       }
+               }
+       }
 }