Skip to content

Commit 0b3c8d7

Browse files
authored
chore: Apply consistent border rounding to inline label (#3382)
1 parent b8815cf commit 0b3c8d7

File tree

2 files changed

+6
-13
lines changed

2 files changed

+6
-13
lines changed

src/select/parts/styles.scss

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,11 @@ $inlineLabel-border-radius: 2px;
132132
}
133133

134134
.inline-label {
135-
background: awsui.$color-background-input-default;
135+
background: awsui.$color-background-container-header;
136+
border-start-start-radius: $inlineLabel-border-radius;
137+
border-start-end-radius: $inlineLabel-border-radius;
138+
border-end-start-radius: $inlineLabel-border-radius;
139+
border-end-end-radius: $inlineLabel-border-radius;
136140
box-sizing: border-box;
137141
display: inline-block;
138142
color: awsui.$color-text-form-label;
@@ -147,14 +151,6 @@ $inlineLabel-border-radius: 2px;
147151
padding-inline: awsui.$space-scaled-xxs;
148152
max-inline-size: calc(100% - (2 * awsui.$space-field-horizontal));
149153
z-index: 1;
150-
151-
&.inline-label-disabled {
152-
background: awsui.$color-background-container-header;
153-
border-start-start-radius: $inlineLabel-border-radius;
154-
border-start-end-radius: $inlineLabel-border-radius;
155-
border-end-start-radius: $inlineLabel-border-radius;
156-
border-end-end-radius: $inlineLabel-border-radius;
157-
}
158154
}
159155

160156
.disabled-reason-tooltip {

src/select/parts/trigger.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -129,10 +129,7 @@ const Trigger = React.forwardRef(
129129
<>
130130
{inlineLabelText ? (
131131
<div className={styles['inline-label-wrapper']}>
132-
<label
133-
htmlFor={controlId}
134-
className={clsx(styles['inline-label'], disabled && styles['inline-label-disabled'])}
135-
>
132+
<label htmlFor={controlId} className={styles['inline-label']}>
136133
{inlineLabelText}
137134
</label>
138135
<div className={styles['inline-label-trigger-wrapper']}>{triggerButton}</div>

0 commit comments

Comments
 (0)