[UIK-5586] Update Select and DropdownMenu styles#3065
Conversation
| display: inline-flex; | ||
| align-items: center; | ||
| justify-content: center; | ||
| gap: var(--intergalactic-spacing-content-gap-medium, 6px); |
There was a problem hiding this comment.
thought it would be more convenient to set it as gap instead of margins for addons
github-actions
Bot
added
base-trigger
select
time-picker
feature-highlight
add-filter
date-picker
labels
| height: var(--intergalactic-spacing-3x, 12px); | ||
| color: var(--intergalactic-icon-non-interactive, oklch(0.094 0.024 153.9 / 0.503)); | ||
| position: relative; | ||
| left: var(--intergalactic-spacing-05x, 2px); |
There was a problem hiding this comment.
we want to shift the chevron optically 2px closer to the right border
There was a problem hiding this comment.
(additional 2px gap that appears before the chevron is ok)
| SBaseTrigger[size='m'] { | ||
| min-width: var(--intergalactic-form-control-m, 32px); | ||
| height: var(--intergalactic-form-control-m, 32px); | ||
| border-radius: var(--intergalactic-control-rounded, 6px); |
There was a problem hiding this comment.
I moved styles that are common for both sizes to the common selector, left here only styles that differ in M/L
| font-size: var(--intergalactic-fs-200, 14px); | ||
|
|
||
| & SAddon { | ||
| &:not(:only-child):first-child { |
There was a problem hiding this comment.
replaced this with gap for the wrapper
| margin-left: var(--intergalactic-spacing-2x, 8px); | ||
| } | ||
|
|
||
| &:only-child { |
There was a problem hiding this comment.
moved this to the SAddon selector
| pointer-events: none; | ||
| } | ||
|
|
||
| SBaseAddon { |
There was a problem hiding this comment.
removed SBaseAddon selector because I don't think it's used anywhere anymore
There was a problem hiding this comment.
SButtonAddon is used for chevron instead (in button trigger)
sheila-semrush
changed the title
sheila-semrush
changed the title
1 participant
Changelog
@semcore/_$component_name
Fixed
Motivation and Context
How has this been tested?
Screenshots (if appropriate):
Types of changes
Checklist: