From 46ca6ce717f54cdc731708e65b1674785e8df1ac Mon Sep 17 00:00:00 2001 From: Kristiyan Kostadinov Date: Thu, 5 Apr 2018 17:04:11 +0200 Subject: [PATCH] fix(table,sort,paginator): incorrect spacing in rtl (#10617) Fixes a few instances in the `table`, `sort` and `paginator` modules where the spacings weren't being inverted in RTL layouts. --- src/lib/paginator/paginator.scss | 5 +++++ src/lib/sort/sort-header.scss | 9 +++++++-- src/lib/table/table.scss | 10 ++++++++++ 3 files changed, 22 insertions(+), 2 deletions(-) diff --git a/src/lib/paginator/paginator.scss b/src/lib/paginator/paginator.scss index 6cafb8d9578..e604d7158d6 100644 --- a/src/lib/paginator/paginator.scss +++ b/src/lib/paginator/paginator.scss @@ -45,6 +45,11 @@ $mat-paginator-button-last-increment-icon-margin: 9px; display: flex; align-items: baseline; margin-right: $mat-paginator-page-size-margin-right; + + [dir='rtl'] & { + margin-right: 0; + margin-left: $mat-paginator-page-size-margin-right; + } } .mat-paginator-page-size-label { diff --git a/src/lib/sort/sort-header.scss b/src/lib/sort/sort-header.scss index a17a2d6d505..1859a68c60e 100644 --- a/src/lib/sort/sort-header.scss +++ b/src/lib/sort/sort-header.scss @@ -34,11 +34,16 @@ $mat-sort-header-arrow-hint-opacity: 0.38; height: $mat-sort-header-arrow-container-size; width: $mat-sort-header-arrow-container-size; min-width: $mat-sort-header-arrow-container-size; - margin: 0 0 0 $mat-sort-header-arrow-margin; position: relative; display: flex; - .mat-sort-header-position-before & { + &, + [dir='rtl'] .mat-sort-header-position-before & { + margin: 0 0 0 $mat-sort-header-arrow-margin; + } + + .mat-sort-header-position-before &, + [dir='rtl'] & { margin: 0 $mat-sort-header-arrow-margin 0 0; } } diff --git a/src/lib/table/table.scss b/src/lib/table/table.scss index 822cb342007..af03f40093c 100644 --- a/src/lib/table/table.scss +++ b/src/lib/table/table.scss @@ -36,10 +36,20 @@ mat-row, mat-header-row { mat-cell:first-child, mat-header-cell:first-child { padding-left: $mat-row-horizontal-padding; + + [dir='rtl'] & { + padding-left: 0; + padding-right: $mat-row-horizontal-padding; + } } mat-cell:last-child, mat-header-cell:last-child { padding-right: $mat-row-horizontal-padding; + + [dir='rtl'] & { + padding-right: 0; + padding-left: $mat-row-horizontal-padding; + } } mat-cell, mat-header-cell {