Skip to content

Commit 08e8349

Browse files
committed
feat(toolbar): use components variable
1 parent a193f0d commit 08e8349

File tree

9 files changed

+18
-240
lines changed

9 files changed

+18
-240
lines changed

core/src/components/button/button.common.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,7 @@ ion-ripple-effect {
293293

294294
:host(.button-solid.in-toolbar:not(.ion-color):not(.in-toolbar-color)) .button-native {
295295
background: #{var(--ion-toolbar-color, var(--background))};
296-
color: #{var(--ion-toolbar-background, var(--color))};
296+
color: #{var(--ion-components-ion-toolbar-bg, (var(--color)))};
297297
}
298298

299299
// Button Badge

core/src/components/button/button.ios.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,5 +291,5 @@
291291

292292
:host(.button-outline.ion-activated.in-toolbar:not(.ion-color):not(.in-toolbar-color)) .button-native {
293293
background: var(--ion-toolbar-color, var(--color));
294-
color: #{var(--ion-toolbar-background, var(--background), ion-color(primary, contrast))};
294+
color: #{var(--ion-components-ion-toolbar-bg, var(--background), ion-color(primary, contrast))};
295295
}

core/src/components/button/button.md.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,6 +240,6 @@
240240
// --------------------------------------------------
241241

242242
:host(.button-outline.ion-activated.in-toolbar:not(.ion-color):not(.in-toolbar-color)) .button-native {
243-
background: var(--ion-toolbar-background, var(--color));
243+
background: var(--ion-components-ion-toolbar-bg, var(--color));
244244
color: #{var(--ion-toolbar-color, var(--background), ion-color(primary, contrast))};
245245
}

core/src/components/buttons/buttons.md.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@
7878
// --------------------------------------------------
7979

8080
::slotted(*) .button-solid {
81-
--color: #{$toolbar-md-background};
81+
--color: var(--ion-components-ion-toolbar-bg);
8282
--background: #{$toolbar-md-color};
8383
--background-activated: transparent;
8484
--background-focused: currentColor;

core/src/components/toolbar/test/basic/index.html

Lines changed: 8 additions & 232 deletions
Original file line numberDiff line numberDiff line change
@@ -17,238 +17,14 @@
1717
<body>
1818
<ion-app>
1919
<ion-header>
20-
<ion-toolbar>
21-
<ion-title> Toolbar </ion-title>
22-
</ion-toolbar>
23-
24-
<ion-toolbar>
25-
<ion-title> This is the title that never ends. It just goes on and on my friend. </ion-title>
26-
</ion-toolbar>
27-
28-
<ion-toolbar>
29-
<ion-buttons slot="secondary">
30-
<ion-button>
31-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
32-
</ion-button>
33-
<ion-button>
34-
<ion-icon slot="icon-only" name="search"></ion-icon>
35-
</ion-button>
36-
</ion-buttons>
37-
<ion-buttons slot="primary">
38-
<ion-button color="secondary">
39-
<ion-icon slot="icon-only" ios="ellipsis-horizontal" md="ellipsis-vertical"></ion-icon>
40-
</ion-button>
41-
</ion-buttons>
42-
<ion-title> Toolbar </ion-title>
43-
</ion-toolbar>
44-
45-
<ion-toolbar>
46-
<ion-buttons slot="start">
47-
<ion-button>
48-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
49-
</ion-button>
50-
<ion-button>
51-
<ion-icon slot="icon-only" name="search"></ion-icon>
52-
</ion-button>
53-
</ion-buttons>
54-
<ion-title> Toolbar with just start button </ion-title>
55-
</ion-toolbar>
56-
57-
<ion-toolbar>
58-
<ion-buttons slot="end">
59-
<ion-button color="secondary">
60-
<ion-icon slot="icon-only" ios="ellipsis-horizontal" md="ellipsis-vertical"></ion-icon>
61-
</ion-button>
62-
</ion-buttons>
63-
<ion-title> Toolbar with just end button </ion-title>
64-
</ion-toolbar>
65-
66-
<ion-toolbar>
67-
<ion-buttons slot="start">
68-
<ion-button>
69-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
70-
</ion-button>
71-
<ion-button>
72-
<ion-icon slot="icon-only" name="search"></ion-icon>
73-
</ion-button>
74-
<ion-button>
75-
<ion-icon slot="icon-only" name="search"></ion-icon>
76-
</ion-button>
77-
</ion-buttons>
78-
<ion-text slot="end"> Small </ion-text>
79-
<ion-title> Toolbar with start content that's long and end content that's short </ion-title>
80-
</ion-toolbar>
81-
82-
<ion-toolbar>
83-
<ion-text slot="start"> Small </ion-text>
84-
<ion-buttons slot="end">
85-
<ion-button>
86-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
87-
</ion-button>
88-
<ion-button>
89-
<ion-icon slot="icon-only" name="search"></ion-icon>
90-
</ion-button>
91-
<ion-button>
92-
<ion-icon slot="icon-only" name="search"></ion-icon>
93-
</ion-button>
94-
</ion-buttons>
95-
<ion-title> Toolbar with start content that's long and end content that's short </ion-title>
96-
</ion-toolbar>
97-
98-
<ion-toolbar>
99-
<ion-buttons slot="secondary">
100-
<ion-button>
101-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
102-
</ion-button>
103-
<ion-button>
104-
<ion-icon slot="icon-only" name="search"></ion-icon>
105-
</ion-button>
106-
</ion-buttons>
107-
<ion-buttons slot="primary">
108-
<ion-button color="secondary">
109-
<ion-icon slot="icon-only" ios="ellipsis-horizontal" md="ellipsis-vertical"></ion-icon>
110-
</ion-button>
111-
</ion-buttons>
112-
<ion-title> This is a long title with buttons. It just goes on and on my friend. </ion-title>
113-
</ion-toolbar>
114-
115-
<ion-toolbar>
116-
<ion-buttons slot="start">
117-
<ion-button>
118-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
119-
</ion-button>
120-
<ion-button>
121-
<ion-icon slot="icon-only" name="search"></ion-icon>
122-
</ion-button>
123-
</ion-buttons>
124-
<ion-buttons slot="end">
125-
<ion-button color="secondary">
126-
<ion-icon slot="icon-only" ios="ellipsis-horizontal" md="ellipsis-vertical"></ion-icon>
127-
</ion-button>
128-
</ion-buttons>
129-
<ion-title> This is a long title with start and end buttons. It just goes on and on my friend. </ion-title>
130-
</ion-toolbar>
131-
132-
<ion-toolbar>
133-
<ion-buttons slot="secondary">
134-
<ion-button>
135-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
136-
</ion-button>
137-
<ion-button>
138-
<ion-icon slot="icon-only" name="search"></ion-icon>
139-
</ion-button>
140-
</ion-buttons>
141-
<ion-buttons slot="primary">
142-
<ion-button color="secondary">
143-
<ion-icon slot="icon-only" ios="ellipsis-horizontal" md="ellipsis-vertical"></ion-icon>
144-
</ion-button>
145-
</ion-buttons>
146-
<ion-title>Defaults</ion-title>
147-
</ion-toolbar>
148-
149-
<ion-toolbar>
150-
<ion-buttons slot="secondary">
151-
<ion-button class="ion-activated">
152-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
153-
</ion-button>
154-
<ion-button class="ion-activated">
155-
<ion-icon slot="icon-only" name="search"></ion-icon>
156-
</ion-button>
157-
</ion-buttons>
158-
<ion-buttons slot="primary">
159-
<ion-button color="secondary" class="ion-activated">
160-
<ion-icon slot="icon-only" ios="ellipsis-horizontal" md="ellipsis-vertical"></ion-icon>
161-
</ion-button>
162-
</ion-buttons>
163-
<ion-title>Defaults.activated</ion-title>
164-
</ion-toolbar>
165-
166-
<ion-toolbar>
167-
<ion-buttons slot="secondary">
168-
<ion-button fill="solid">
169-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
170-
</ion-button>
171-
<ion-button fill="solid">
172-
<ion-icon slot="start" name="person-circle"></ion-icon>
173-
Solid
174-
</ion-button>
175-
</ion-buttons>
176-
<ion-title>Solid</ion-title>
177-
<ion-buttons slot="primary">
178-
<ion-button fill="solid" color="secondary">
179-
Help
180-
<ion-icon slot="end" name="help-circle"></ion-icon>
181-
</ion-button>
182-
</ion-buttons>
183-
</ion-toolbar>
184-
185-
<ion-toolbar>
186-
<ion-buttons slot="secondary">
187-
<ion-button href="#" fill="solid" class="ion-activated">
188-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
189-
</ion-button>
190-
<ion-button fill="solid" class="ion-activated">
191-
<ion-icon slot="start" name="person-circle"></ion-icon>
192-
Solid
193-
</ion-button>
194-
</ion-buttons>
195-
<ion-title>Solid Activated</ion-title>
196-
<ion-buttons slot="primary">
197-
<ion-button fill="solid" color="secondary" class="ion-activated">
198-
Help
199-
<ion-icon slot="end" name="help-circle"></ion-icon>
200-
</ion-button>
201-
</ion-buttons>
202-
</ion-toolbar>
203-
204-
<ion-toolbar>
205-
<ion-buttons slot="secondary">
206-
<ion-button fill="outline">
207-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
208-
</ion-button>
209-
<ion-button fill="outline">
210-
<ion-icon slot="start" name="star"></ion-icon>
211-
Star
212-
</ion-button>
213-
</ion-buttons>
214-
<ion-buttons slot="primary">
215-
<ion-button color="secondary" fill="outline">
216-
Info
217-
<ion-icon slot="end" name="information-circle"></ion-icon>
218-
</ion-button>
219-
</ion-buttons>
220-
<ion-title>Outline</ion-title>
221-
</ion-toolbar>
222-
223-
<ion-toolbar>
224-
<ion-buttons slot="secondary">
225-
<ion-button fill="outline" class="ion-activated">
226-
<ion-icon slot="icon-only" name="person-circle"></ion-icon>
227-
</ion-button>
228-
<ion-button fill="outline" class="ion-activated">
229-
<ion-icon slot="start" name="star"></ion-icon>
230-
Star
231-
</ion-button>
232-
</ion-buttons>
233-
<ion-buttons slot="primary">
234-
<ion-button color="secondary" fill="outline" class="ion-activated">
235-
Info
236-
<ion-icon slot="end" name="information-circle"></ion-icon>
237-
</ion-button>
238-
</ion-buttons>
239-
<ion-title>Outline.activated</ion-title>
240-
</ion-toolbar>
241-
242-
<ion-toolbar>
243-
<ion-buttons slot="secondary">
244-
<ion-button> Go Back </ion-button>
245-
</ion-buttons>
246-
<ion-buttons slot="primary">
247-
<ion-button href="#"> Edit </ion-button>
248-
</ion-buttons>
249-
<ion-title>Text Only</ion-title>
250-
</ion-toolbar>
251-
</ion-header>
20+
<ion-toolbar>
21+
<ion-button fill="solid">
22+
<ion-icon slot="start" name="person-circle"></ion-icon>
23+
Solid
24+
</ion-button>
25+
<span>Solid</span>
26+
</ion-toolbar>
27+
</ion-header>
25228

25329
<ion-content id="content" fullscreen="true" class="ion-padding">
25430
<h1>Content</h1>

core/src/components/toolbar/toolbar.md.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
// --------------------------------------------------
66

77
:host {
8-
--background: #{$toolbar-md-background};
8+
--background: var(--ion-components-ion-toolbar-bg);
99
--color: #{$toolbar-md-color};
1010
--border-color: var(--ion-components-ion-toolbar-border-color);
1111
--padding-top: 0;

core/src/css/palettes/dark.scss

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,7 @@ $colors: (
135135
--ion-text-color-step-850: #262626;
136136
--ion-text-color-step-900: #1a1a1a;
137137
--ion-text-color-step-950: #0d0d0d;
138+
// TODO(FW-4004): Implement the following components to the dark theme in ionic modular
138139
--ion-item-background: #000000;
139140
--ion-card-background: #1c1c1d;
140141
}
@@ -184,6 +185,7 @@ $colors: (
184185
--ion-text-color-step-850: #363636;
185186
--ion-text-color-step-900: #2a2a2a;
186187
--ion-text-color-step-950: #1e1e1e;
188+
// TODO(FW-4004): Implement the following components to the dark theme in ionic modular
187189
--ion-item-background: #1e1e1e;
188190
--ion-toolbar-background: #1f1f1f;
189191
--ion-tab-bar-background: #1f1f1f;

core/src/themes/ios/dark.tokens.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ export const darkTheme: DarkTheme = {
2323
background: '#1c1c1d',
2424
},
2525
IonItem: {
26-
background: '#000000',
26+
bg: '#000000',
2727
},
2828
IonModal: {
2929
bg: 'var(--ion-color-gray-100)',

core/src/themes/md/dark.tokens.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,13 @@ export const darkTheme: DarkTheme = {
2222

2323
components: {
2424
IonCard: {
25-
background: '#1e1e1e',
25+
bg: '#1e1e1e',
2626
},
2727
IonItem: {
2828
background: '#1e1e1e',
2929
},
3030
IonToolbar: {
31-
background: '#1f1f1f',
31+
bg: '#1f1f1f',
3232
},
3333
IonTabBar: {
3434
background: '#1f1f1f',

0 commit comments

Comments
 (0)