Skip to content

Commit 5fb1e08

Browse files
committed
fix(toolbar): add a min-width to the toolbar content so that it won't overlap buttons
fixes #5657
1 parent 5ca2a55 commit 5fb1e08

File tree

3 files changed

+19
-1
lines changed

3 files changed

+19
-1
lines changed

ionic/components/toolbar/test/scenarios/main.html

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,22 @@
22
<ion-title>This is the title that never ends. It just goes on and on my friend.</ion-title>
33
</ion-toolbar>
44

5+
<ion-toolbar>
6+
<ion-buttons start>
7+
<button>
8+
<ion-icon name="contact"></ion-icon>
9+
</button>
10+
<button>
11+
<ion-icon name="search"></ion-icon>
12+
</button>
13+
</ion-buttons>
14+
<ion-buttons end>
15+
<button secondary>
16+
<ion-icon name="more"></ion-icon>
17+
</button>
18+
</ion-buttons>
19+
<ion-title>This is a long title with buttons. It just goes on and on my friend.</ion-title>
20+
</ion-toolbar>
521

622
<ion-toolbar>
723
<ion-buttons start>
@@ -207,4 +223,4 @@
207223
.toolbar {
208224
border-bottom: 1px solid black;
209225
}
210-
</style>
226+
</style>

ionic/components/toolbar/toolbar.ios.scss

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ ion-navbar-section {
5757
.toolbar-content {
5858
flex: 1;
5959
order: map-get($toolbar-order-ios, content);
60+
min-width: 0;
6061
}
6162

6263
.toolbar-title {

ionic/components/toolbar/toolbar.md.scss

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ ion-navbar-section {
5252
flex: 1;
5353
order: map-get($toolbar-order-md, content);
5454
max-width: 100%;
55+
min-width: 0;
5556
}
5657

5758
.toolbar-title {

0 commit comments

Comments
 (0)