Skip to content

Commit da7615a

Browse files
authored
fix(ui5-toolbar-button): add css for ui5-toolbar-button for left alignment (#11913) (#11914)
* fix(ui5-toolbar-button): add css for ui5-toolbar-button for left alignment (#11913) * fix(ui5-toolbar-button): add padding for icon-only button in popover
1 parent 534e64c commit da7615a

File tree

4 files changed

+12
-3
lines changed

4 files changed

+12
-3
lines changed

packages/main/src/ToolbarButton.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ import type ButtonDesign from "./types/ButtonDesign.js";
88
import ToolbarItem from "./ToolbarItem.js";
99
import ToolbarButtonTemplate from "./ToolbarButtonTemplate.js";
1010

11+
import ToolbarButtonCss from "./generated/themes/ToolbarButton.css.js";
12+
1113
type ToolbarButtonAccessibilityAttributes = ButtonAccessibilityAttributes;
1214

1315
/**
@@ -29,6 +31,7 @@ type ToolbarButtonAccessibilityAttributes = ButtonAccessibilityAttributes;
2931
tag: "ui5-toolbar-button",
3032
template: ToolbarButtonTemplate,
3133
renderer: jsxRenderer,
34+
styles: [ToolbarButtonCss],
3235
})
3336

3437
/**
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
.ui5-tb-popover-item.ui5-tb-button::part(button) {
2+
justify-content: start;
3+
}
4+
5+
.ui5-tb-popover-item.ui5-tb-button[icon-only]::part(button) {
6+
padding: 0 var(--_ui5_button_base_padding);
7+
}

packages/main/src/themes/ToolbarButtonPopover.css

Lines changed: 0 additions & 3 deletions
This file was deleted.

packages/main/test/pages/Toolbar.html

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,8 @@
152152
<ui5-toolbar-separator></ui5-toolbar-separator>
153153
<ui5-toolbar-button icon="employee" text="Call me later" stable-dom-ref="tb-button-employee-d"
154154
overflow-priority="AlwaysOverflow"></ui5-toolbar-button>
155+
<ui5-toolbar-button icon="decline" text="Longer for alignment check"
156+
stable-dom-ref="tb-button-employee-d" overflow-priority="AlwaysOverflow"></ui5-toolbar-button>
155157
</ui5-toolbar>
156158
</section>
157159

0 commit comments

Comments
 (0)