Skip to content

Conversation

@natemoo-re
Copy link
Member

#101654 was regretfully merged without a (prefers-reduced-motion: reduce) check. This PR implements that check to remove CSS animations and also the JS-driven bleep animation.

splash-loader-reduced-motion

@natemoo-re natemoo-re requested review from a team November 17, 2025 18:19
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 17, 2025
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: SPA Loader Breaks Reduced Motion Accessibility

The @media (prefers-reduced-motion: reduce) CSS rule is missing from the SVG <style> block. The corresponding template file (src/sentry/templates/sentry/partial/loader.html) includes this rule to disable CSS animations for users preferring reduced motion, but static/index.ejs does not. This breaks the accessibility feature on the SPA loader path.

static/index.ejs#L55-L71

sentry/static/index.ejs

Lines 55 to 71 in f0ec402

--stagger: 50ms;
}
path {
stroke-dashoffset: 1150;
stroke-dasharray: 1150;
animation: draw calc(var(--duration) + (var(--i) * var(--stagger)) + (1.3s - var(--delay))) calc(var(--delay) + ((3 - var(--i)) * var(--stagger))) ease-out;
}
[data-index="0"] {
stroke: #F0369A;
}
[data-index="1"] {
stroke: #67C800;
}
[data-index="2"] {
stroke: #7553FF;
}
@keyframes draw {

Fix in Cursor Fix in Web


@natemoo-re natemoo-re enabled auto-merge (squash) November 17, 2025 18:27
@natemoo-re natemoo-re merged commit 688e214 into master Nov 17, 2025
64 checks passed
@natemoo-re natemoo-re deleted the ui2/nm/splash-reduced-motion branch November 17, 2025 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants