Skip to content

Commit

Permalink
fix(checkbox): Remove RTL styles from checkbox ripple (#5134)
Browse files Browse the repository at this point in the history
  • Loading branch information
abhiomkar authored Oct 1, 2019
1 parent 199534d commit a646516
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 7 deletions.
8 changes: 3 additions & 5 deletions packages/mdc-checkbox/_mixins.scss
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@
@import "@material/feature-targeting/functions";
@import "@material/feature-targeting/mixins";
@import "@material/ripple/mixins";
@import "@material/rtl/mixins";
@import "@material/touch-target/mixins";
@import "./functions";
@import "./keyframes";
Expand Down Expand Up @@ -61,7 +60,7 @@ $mdc-checkbox-ripple-target: ".mdc-checkbox__ripple";
}

@include mdc-checkbox-focus-indicator-color($mdc-checkbox-baseline-theme-color, $query: $query);
@include mdc-checkbox-density($mdc-checkbox-density-default-scale, $query: $query);
@include mdc-checkbox-density($mdc-checkbox-density-scale, $query: $query);
}

@include mdc-checkbox-container-colors($query: $query);
Expand Down Expand Up @@ -217,7 +216,7 @@ $mdc-checkbox-ripple-target: ".mdc-checkbox__ripple";
/// Sets density scale for checkbox.
///
/// @param {Number | String} $density-scale - Density scale value for component. Supported density scale values
/// `-3`, `-2`, `-1`, `0`.
/// `-4`, `-3`, `-2`, `-1`, `0`.
///
@mixin mdc-checkbox-density($density-scale, $query: mdc-feature-all()) {
$size: mdc-density-prop-value(
Expand All @@ -239,9 +238,8 @@ $mdc-checkbox-ripple-target: ".mdc-checkbox__ripple";

.mdc-checkbox__background {
@include mdc-feature-targets($feat-structure) {
@include mdc-rtl-reflexive-position(left, $checkbox-padding, ".mdc-checkbox");

top: $checkbox-padding;
left: $checkbox-padding;
}
}

Expand Down
2 changes: 1 addition & 1 deletion packages/mdc-checkbox/_variables.scss
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ $mdc-checkbox-focus-indicator-opacity: map-get($mdc-ripple-dark-ink-opacities, f

$mdc-checkbox-minimum-size: 28px !default;
$mdc-checkbox-maximum-size: $mdc-checkbox-ripple-size !default;
$mdc-checkbox-density-default-scale: $mdc-density-default-scale !default;
$mdc-checkbox-density-scale: $mdc-density-default-scale !default;
$mdc-checkbox-density-config: (
size: (
minimum: $mdc-checkbox-minimum-size,
Expand Down
1 change: 0 additions & 1 deletion packages/mdc-checkbox/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@
"@material/dom": "^3.1.0",
"@material/feature-targeting": "^3.1.0",
"@material/ripple": "^3.2.0",
"@material/rtl": "^3.2.0",
"@material/theme": "^3.1.0",
"@material/touch-target": "^0.0.0",
"tslib": "^1.9.3"
Expand Down

0 comments on commit a646516

Please sign in to comment.