Skip to content

Commit f6854db

Browse files
aweellMarcosld
andauthored
feat(Select, Search, Autocomplete): Align menu item styles and fixes (#1484)
Telefonica/mistica-design#2517 --------- Co-authored-by: Marcos Luis Delgado <mluis@tuenti.com>
1 parent 65ed0b2 commit f6854db

10 files changed

Lines changed: 17 additions & 9 deletions
Loading
-1.56 KB
Loading
-441 Bytes
Loading
-1.64 KB
Loading
Loading
Loading
Loading

src/autocomplete.css.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@ import {style} from '@vanilla-extract/css';
22
import {sprinkles} from './sprinkles.css';
33
import {vars, vars as skinVars} from './skins/skin-contract.css';
44

5+
export const optionListContainerPadding = 8;
6+
57
export const autocompleteContainer = style([
68
sprinkles({
79
position: 'relative',
@@ -11,7 +13,7 @@ export const autocompleteContainer = style([
1113
export const optionsList = style([
1214
sprinkles({
1315
position: 'absolute',
14-
padding: 8,
16+
padding: optionListContainerPadding,
1517
borderRadius: vars.borderRadii.popup,
1618
}),
1719
{
@@ -31,7 +33,7 @@ export const optionBaseItem = style([
3133
paddingY: 12,
3234
}),
3335
{
34-
borderRadius: skinVars.borderRadii.popup,
36+
borderRadius: `calc(${skinVars.borderRadii.popup} - ${optionListContainerPadding / 2}px)`,
3537
height: 48,
3638
userSelect: 'none',
3739
display: 'block',

src/select.css.ts

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ export const vars = {
2727
};
2828

2929
export const chevronSize = pxToRem(20);
30+
export const optionContainerPadding = 8;
3031

3132
export const optionsContainer = style([
3233
sprinkles({
@@ -41,10 +42,8 @@ export const optionsContainer = style([
4142
top,
4243
left,
4344
minWidth,
44-
boxShadow:
45-
'0px 5px 5px -3px rgba(0,0,0,0.2), 0px 8px 10px 1px rgba(0,0,0,0.14), 0px 3px 14px 2px rgba(0,0,0,0.12)',
46-
paddingTop: 8,
47-
paddingBottom: 8,
45+
boxShadow: '0px 2px 4px 0px #00000033',
46+
padding: optionContainerPadding,
4847
transformOrigin,
4948
transition: 'opacity .03s linear,transform .12s cubic-bezier(0,0,.2,1) .15s',
5049
overflowY: 'auto',
@@ -208,16 +207,20 @@ export const menuItem = style([
208207
alignItems: 'center',
209208
cursor: 'pointer',
210209
paddingY: 8,
211-
paddingX: 16,
210+
paddingX: 8,
212211
}),
212+
{
213+
borderRadius: `calc(${skinVars.borderRadii.popup} - ${optionContainerPadding / 2}px)`,
214+
},
215+
213216
{
214217
height: pxToRem(48),
215218
transition: 'background-color 150ms cubic-bezier(0.4, 0, 0.2, 1) 0ms',
216219

217220
'@media': {
218221
[mq.supportsHover]: {
219222
':hover': {
220-
backgroundColor: 'rgba(0, 0, 0, 0.08)',
223+
backgroundColor: skinVars.colors.backgroundContainerHover,
221224
},
222225
},
223226
},

src/text-field-base.css.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,8 @@ const helperTextDesktopLineHeight = createVar();
3535
const labelScaleDesktop = createVar();
3636
const labelScaleMobile = createVar();
3737

38+
export const suggestionsContainerPadding = 8;
39+
3840
export const fieldVars = {
3941
mobileFontSize,
4042
desktopFontSize,
@@ -363,6 +365,7 @@ export const menuItemBase = style([
363365
minHeight: pxToRem(48),
364366
padding: '6px 8px',
365367
userSelect: 'none',
368+
borderRadius: `calc(${skinVars.borderRadii.popup} - ${suggestionsContainerPadding / 2}px)`,
366369
},
367370
]);
368371

@@ -397,7 +400,7 @@ export const suggestionsContainer = style([
397400
marginTop: 8,
398401
boxSizing: 'border-box',
399402
boxShadow: '0px 2px 4px 0px #00000033',
400-
padding: 8,
403+
padding: suggestionsContainerPadding,
401404
background: skinVars.colors.backgroundContainer,
402405
borderRadius: skinVars.borderRadii.popup,
403406

0 commit comments

Comments
 (0)