Skip to content

Commit dda2632

Browse files
authored
fix: keep button radius and allow more space for focus outline (#10162)
1 parent 06d5f48 commit dda2632

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

packages/menu-bar/src/styles/vaadin-menu-bar-base-styles.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,33 +17,34 @@ export const menuBarStyles = css`
1717
[part='container'] {
1818
display: flex;
1919
flex-wrap: nowrap;
20-
margin: calc((var(--vaadin-focus-ring-width) + 1px) * -1);
20+
margin: calc((var(--vaadin-focus-ring-width) + 2px) * -1);
2121
overflow: hidden;
22-
padding: calc(var(--vaadin-focus-ring-width) + 1px);
22+
padding: calc(var(--vaadin-focus-ring-width) + 2px);
2323
position: relative;
2424
width: 100%;
2525
--_gap: var(--vaadin-menu-bar-gap, 0px);
2626
--_bw: var(--vaadin-button-border-width, 1px);
2727
gap: var(--_gap);
2828
--_rad-button: var(--vaadin-button-border-radius, var(--vaadin-radius-m));
29-
--_rad: min(var(--_gap) * 1000, var(--_rad-button));
3029
}
3130
3231
::slotted(vaadin-menu-bar-button:not(:first-of-type)) {
3332
margin-inline-start: min(var(--_bw) * -1 + var(--_gap) * 1000, 0px);
3433
}
3534
3635
::slotted(vaadin-menu-bar-button) {
37-
border-radius: var(--_rad);
36+
border-radius: 0;
3837
}
3938
4039
::slotted([first-visible]),
41-
:host([has-single-button]) ::slotted([slot='overflow']) {
40+
:host([has-single-button]) ::slotted([slot='overflow']),
41+
::slotted(vaadin-menu-bar-button[theme~='tertiary']) {
4242
border-start-start-radius: var(--_rad-button);
4343
border-end-start-radius: var(--_rad-button);
4444
}
4545
46-
::slotted(:is([last-visible], [slot='overflow'])) {
46+
::slotted(:is([last-visible], [slot='overflow'])),
47+
::slotted(vaadin-menu-bar-button[theme~='tertiary']) {
4748
border-start-end-radius: var(--_rad-button);
4849
border-end-end-radius: var(--_rad-button);
4950
}

0 commit comments

Comments
 (0)