Skip to content

Commit f816a29

Browse files
chore: apply suggestions from code review
Co-authored-by: Maria Hutt <[email protected]>
1 parent 0865a33 commit f816a29

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

core/src/components/button/button.ionic.scss

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -267,6 +267,8 @@
267267
font-size: globals.$ion-font-size-500;
268268
}
269269

270+
:host(.button-xsmall) ::slotted(ion-icon[slot="start"]),
271+
:host(.button-xsmall) ::slotted(ion-spinner[slot="start"]),
270272
:host(.button-small) ::slotted(ion-icon[slot="start"]),
271273
:host(.button-small) ::slotted(ion-spinner[slot="start"]) {
272274
@include globals.margin-horizontal(null, globals.$ion-space-200);
@@ -277,14 +279,16 @@
277279
@include globals.margin-horizontal(null, globals.$ion-space-250);
278280
}
279281

280-
:host(.button-large) ::slotted(ion-icon[slot="start"]) {
281-
@include globals.margin-horizontal(null, globals.$ion-space-300);
282-
}
283-
282+
:host(.button-large) ::slotted(ion-icon[slot="start"]),
283+
:host(.button-large) ::slotted(ion-spinner[slot="start"]),
284+
:host(.button-xlarge) ::slotted(ion-icon[slot="start"]),
284285
:host(.button-xlarge) ::slotted(ion-spinner[slot="start"]) {
285286
@include globals.margin-horizontal(null, globals.$ion-space-300);
286287
}
287288

289+
290+
:host(.button-xsmall) ::slotted(ion-icon[slot="end"]),
291+
:host(.button-xsmall) ::slotted(ion-spinner[slot="end"]),
288292
:host(.button-small) ::slotted(ion-icon[slot="end"]),
289293
:host(.button-small) ::slotted(ion-spinner[slot="end"]) {
290294
@include globals.margin-horizontal(globals.$ion-space-200, null);
@@ -295,10 +299,10 @@
295299
@include globals.margin-horizontal(globals.$ion-space-250, null);
296300
}
297301

298-
:host(.button-large) ::slotted(ion-icon[slot="end"]) {
299-
@include globals.margin-horizontal(globals.$ion-space-300, null);
300-
}
301-
302+
:host(.button-large) ::slotted(ion-icon[slot="end"]),
303+
:host(.button-large) ::slotted(ion-spinner[slot="end"]),
304+
:host(.button-xlarge) ::slotted(ion-icon[slot="end"]),
302305
:host(.button-xlarge) ::slotted(ion-spinner[slot="end"]) {
303306
@include globals.margin-horizontal(globals.$ion-space-300, null);
304307
}
308+

core/src/components/button/test/spinner/button.e2e.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { expect } from '@playwright/test';
22
import { configs, test } from '@utils/test/playwright';
33

4-
configs({ modes: ['ionic-md', 'ionic-ios'] }).forEach(({ config, screenshot, title }) => {
4+
configs({ modes: ['ionic-md'], directions: ['ltr'] }).forEach(({ config, screenshot, title }) => {
55
test.describe(title('button: spinner'), () => {
66
test('should not have visual regressions', async ({ page }) => {
77
await page.goto(`/src/components/button/test/spinner`, config);

0 commit comments

Comments
 (0)