Skip to content

Add reduced motion accessibility support#107

Closed
jaygaikar-09 wants to merge 4 commits into
komalharshita:mainfrom
jaygaikar-09:fix/reduced-motion-accessibility
Closed

Add reduced motion accessibility support#107
jaygaikar-09 wants to merge 4 commits into
komalharshita:mainfrom
jaygaikar-09:fix/reduced-motion-accessibility

Conversation

@jaygaikar-09

@jaygaikar-09 jaygaikar-09 commented May 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Added reduced-motion accessibility support using a prefers-reduced-motion media query in static/style.css. This update reduces animations and transitions for users who enable reduced motion in their operating system accessibility settings while preserving the existing layout and default animations for other users.

Closes #80


Type of Change

  • Bug fix
  • Feature
  • Data
  • Documentation
  • Style
  • Refactor
  • Test

What Was Changed

File Change made
static/style.css Added prefers-reduced-motion media query to reduce animations, transitions, and smooth scrolling for accessibility

How to Test

  1. Run the app:
python app.py

@vercel

vercel Bot commented May 15, 2026

Copy link
Copy Markdown

@jaygaikar-09 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

@komalharshita

Copy link
Copy Markdown
Owner

@jaygaikar-09 do resolve the merge conflicts to merge the PR

@komalharshita komalharshita added the need review Further information is requested label May 17, 2026
@jaygaikar-09

Copy link
Copy Markdown
Contributor Author

@Harshada-Yele Done!!

@Harshada-Yele

Copy link
Copy Markdown
Contributor

@jaygaikar-09
Sync your branch with the latest main and ensure all checks pass

@jaygaikar-09

Copy link
Copy Markdown
Contributor Author

@Harshada-Yele Done!

@komalharshita komalharshita left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for the contribution and for working on accessibility improvements for DevPath.

However, after review, this PR cannot be merged in its current form because it includes a large number of unrelated CSS removals and modifications beyond the scope of adding reduced-motion support.

The issue specifically targets accessibility improvements using prefers-reduced-motion, but this PR additionally modifies/removes styles related to:

  • navigation
  • mobile menu behavior
  • badges
  • hover effects
  • branding styles
  • shared layout/UI components

These unrelated changes introduce unnecessary regression risk in globally shared styles and make the PR unsafe to merge as an accessibility-focused contribution.

For maintainability and safe review practices, accessibility PRs should remain tightly scoped and avoid unrelated style rewrites or cleanup.

Since the PR scope has diverged significantly from the intended issue and would require substantial restructuring/rework, this PR is being closed rather than merged.

You are welcome to open a new, smaller PR focused only on:

  • prefers-reduced-motion
  • animation/transition overrides
  • motion accessibility improvements directly tied to the issue

Appreciate the effort and understanding.

@komalharshita komalharshita added gssoc:invalid This doesn't seem right and removed type:bug Something isn't working need review Further information is requested type:accessibility ui labels May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:invalid This doesn't seem right gssoc-2026

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add prefers-reduced-motion media query to respect user accessibility settings

3 participants