Skip to content

Commit

Permalink
feat(clay-css): Mixins clay-css adds option to pass in `scrollbar-w…
Browse files Browse the repository at this point in the history
…idth` for future proofing this property. It's available in Firefox 64+

fix(clay-css) Forms `select[multiple]` and `select[size]` should use `scrollbar-width: thin;`

issue #2896
  • Loading branch information
pat270 authored and marcoscv-work committed Mar 6, 2020
1 parent 27241af commit 4fbe1df
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions packages/clay-css/src/scss/mixins/_globals.scss
Original file line number Diff line number Diff line change
Expand Up @@ -204,6 +204,7 @@
resize: map-get($map, resize);
right: map-get($map, right);
scroll-behavior: map-get($map, scroll-behavior);
scrollbar-width: map-get($map, scrollbar-width);
table-layout: map-get($map, table-layout);
text-align: map-get($map, text-align);
text-decoration: map-get($map, text-decoration);
Expand Down
2 changes: 2 additions & 0 deletions packages/clay-css/src/scss/variables/_forms.scss
Original file line number Diff line number Diff line change
Expand Up @@ -348,6 +348,8 @@ $input-select-size: map-deep-merge((
background-image: none,
padding-left: 0.5rem,
padding-right: 0.5rem,
// Future proof: if https://github.com/w3c/csswg-drafts/issues/1958 ever gets added
scrollbar-width: thin,
focus: (
background-image: none,
),
Expand Down

0 comments on commit 4fbe1df

Please sign in to comment.