mirror of
https://github.com/ionic-team/ionic-framework.git
synced 2025-11-10 00:27:41 +08:00
fix(spinner): ensure transform doesn't overwrite circular animation (#24643)
This commit is contained in:
@ -115,7 +115,7 @@ svg {
|
||||
// Spinner: circular
|
||||
// --------------------------------------------------
|
||||
|
||||
:host(.spinner-circular) {
|
||||
:host(.spinner-circular) svg {
|
||||
animation: spinner-circular linear infinite;
|
||||
}
|
||||
|
||||
|
||||
18
core/src/components/spinner/test/transform/e2e.ts
Normal file
18
core/src/components/spinner/test/transform/e2e.ts
Normal file
@ -0,0 +1,18 @@
|
||||
import { newE2EPage } from '@stencil/core/testing';
|
||||
|
||||
// made to catch animation issues when transform is applied to host
|
||||
// example: https://github.com/ionic-team/ionic-framework/issues/19247
|
||||
test('spinner: transform', async () => {
|
||||
const page = await newE2EPage({
|
||||
url: '/src/components/spinner/test/transform?ionic:_testing=true'
|
||||
});
|
||||
|
||||
const compare = await page.compareScreenshot();
|
||||
expect(compare).toMatchScreenshot();
|
||||
|
||||
// wait for a bit less than the spinner's duration (to avoid looping before screenshot is taken)
|
||||
await new Promise(resolve => setTimeout(resolve, 1500));
|
||||
|
||||
const compare2 = await page.compareScreenshot();
|
||||
expect(compare).toMatchScreenshot();
|
||||
});
|
||||
75
core/src/components/spinner/test/transform/index.html
Normal file
75
core/src/components/spinner/test/transform/index.html
Normal file
@ -0,0 +1,75 @@
|
||||
<!DOCTYPE html>
|
||||
<html lang="en" dir="ltr">
|
||||
|
||||
<head>
|
||||
<meta charset="UTF-8">
|
||||
<title>Spinner - Transform</title>
|
||||
<meta name="viewport"
|
||||
content="width=device-width, initial-scale=1.0, minimum-scale=1.0, maximum-scale=1.0, user-scalable=no">
|
||||
<link href="../../../../../css/ionic.bundle.css" rel="stylesheet">
|
||||
<link href="../../../../../scripts/testing/styles.css" rel="stylesheet">
|
||||
<script src="../../../../../scripts/testing/scripts.js"></script>
|
||||
<script nomodule src="../../../../../dist/ionic/ionic.js"></script>
|
||||
<script type="module" src="../../../../../dist/ionic/ionic.esm.js"></script>
|
||||
|
||||
<style>
|
||||
ion-spinner {
|
||||
transform: translate(-10px, -10px);
|
||||
}
|
||||
</style>
|
||||
</head>
|
||||
|
||||
<body>
|
||||
<ion-app>
|
||||
|
||||
<ion-header>
|
||||
<ion-toolbar>
|
||||
<ion-title>Spinner - Transform</ion-title>
|
||||
</ion-toolbar>
|
||||
</ion-header>
|
||||
|
||||
<ion-content>
|
||||
<ion-list>
|
||||
<ion-item>
|
||||
<ion-spinner slot="start" duration="2000" name="bubbles"></ion-spinner>
|
||||
<code>bubbles</code>
|
||||
</ion-item>
|
||||
<ion-item>
|
||||
<ion-spinner slot="start" duration="2000" name="circles"></ion-spinner>
|
||||
<code>circles</code>
|
||||
</ion-item>
|
||||
<ion-item>
|
||||
<ion-spinner slot="start" duration="2000" name="circular"></ion-spinner>
|
||||
<code>circular</code>
|
||||
</ion-item>
|
||||
<ion-item>
|
||||
<ion-spinner slot="start" duration="2000" name="crescent"></ion-spinner>
|
||||
<code>crescent</code>
|
||||
</ion-item>
|
||||
<ion-item>
|
||||
<ion-spinner slot="start" duration="2000" name="dots"></ion-spinner>
|
||||
<code>dots</code>
|
||||
</ion-item>
|
||||
<ion-item>
|
||||
<ion-spinner slot="start" duration="2000" name="lines"></ion-spinner>
|
||||
<code>lines</code>
|
||||
</ion-item>
|
||||
<ion-item>
|
||||
<ion-spinner slot="start" duration="2000" name="lines-sharp"></ion-spinner>
|
||||
<code>lines-sharp</code>
|
||||
</ion-item>
|
||||
<ion-item>
|
||||
<ion-spinner slot="start" duration="2000" name="lines-sharp-small"></ion-spinner>
|
||||
<code>lines-sharp-small</code>
|
||||
</ion-item>
|
||||
<ion-item>
|
||||
<ion-spinner slot="start" duration="2000" name="lines-small"></ion-spinner>
|
||||
<code>lines-small</code>
|
||||
</ion-item>
|
||||
</ion-list>
|
||||
</ion-content>
|
||||
|
||||
</ion-app>
|
||||
</body>
|
||||
|
||||
</html>
|
||||
Reference in New Issue
Block a user