AngeloooooandDraculabo
8ed4593f73
* Refactor Gemini model display: filter old versions, group low/high pro, and improve naming * Fix potential negative minutes and refine Gemini 3 Pro grouping logic based on PR feedback * The account change process has been sped up * Address PR review: enhance error message and parameterize polling interval in process handler * Address PR review: fix potential race condition in database backup logic * fix: prevent renderer white screen and harden sentry gating * feat: localize model labels * chore: enable hoisted node-linker * chore: ignore .agent/.claude/.codex in prettier --------- Co-authored-by: Draculabo <[email protected]>
39 lines
1.1 KiB
Markdown
39 lines
1.1 KiB
Markdown
---
|
|
title: Animate SVG Wrapper Instead of SVG Element
|
|
impact: LOW
|
|
impactDescription: enables hardware acceleration
|
|
tags: rendering, svg, css, animation, performance
|
|
---
|
|
|
|
## Animate SVG Wrapper Instead of SVG Element
|
|
|
|
Many browsers don't have hardware acceleration for CSS3 animations on SVG elements. Wrap SVG in a `<div>` and animate the wrapper instead.
|
|
|
|
**Incorrect (animating SVG directly - no hardware acceleration):**
|
|
|
|
```tsx
|
|
function LoadingSpinner() {
|
|
return (
|
|
<svg className="animate-spin" width="24" height="24" viewBox="0 0 24 24">
|
|
<circle cx="12" cy="12" r="10" stroke="currentColor" />
|
|
</svg>
|
|
);
|
|
}
|
|
```
|
|
|
|
**Correct (animating wrapper div - hardware accelerated):**
|
|
|
|
```tsx
|
|
function LoadingSpinner() {
|
|
return (
|
|
<div className="animate-spin">
|
|
<svg width="24" height="24" viewBox="0 0 24 24">
|
|
<circle cx="12" cy="12" r="10" stroke="currentColor" />
|
|
</svg>
|
|
</div>
|
|
);
|
|
}
|
|
```
|
|
|
|
This applies to all CSS transforms and transitions (`transform`, `opacity`, `translate`, `scale`, `rotate`). The wrapper div allows browsers to use GPU acceleration for smoother animations.
|