Adventure Idea: define intermediate and expert levels for The Accessibility Nightmare - #90
Merged
sinduri-g merged 2 commits intoSep 7, 2026
Conversation
…bility Nightmare - Intermediate (The Checkout Trap): drops the Storybook component-library framing. Three checkout components built from role-free containers, on a page axe-core reports clean before and after the repair. Verified on a React build in development and production across all six reachable states. - Expert (The Compliance Engine): drops Real User Monitoring, OpenTelemetry and Grafana as three levels of scope in one. Now about audit coverage, centred on a keyboard trap in a vendor payment widget that no DOM scanner reports, because a trap is a behavior rather than a property of the markup. - Removes NVDA and VoiceOver, which are Windows and macOS only and cannot run in a Codespace. Replaced by the Virtual Screen Reader, documented as a simulation rather than a substitute for real assistive technology. - Removes Pa11y as redundant with axe-core. - Expert gate runs locally because forks disable Actions by default, not because Actions cannot be triggered from a Codespace. - Reframes the opening around the customers affected rather than the penalties, across the Theme, README, and index.yaml backstory and overview. Overview no longer names tooling. - Beginner: corrects three lines naming tooling the level does not use, screen readers to Playwright, axe DevTools to axe-core, Next.js to Vite. Story, objectives and Lighthouse figure unchanged, implementation untouched. - docs/beginner.yaml gains topics, contributor and meta_description with no existing line modified. Signed-off-by: sindurigf <sinduri.g@gmail.com>
sindurigf
changed the title
KatharinaSick
previously approved these changes
Sep 7, 2026
KatharinaSick
left a comment
KatharinaSick
left a comment
Contributor
There was a problem hiding this comment.
A bit of context: Sinduri will be implementing intermediate & expert of this adventure :)
… the answer - Story gives the symptom only. It previously described the keyboard cycling, which handed the player the diagnosis the level exists to reach. - Objectives return to the original three, multi-step flows, production monitoring and a compliance report, with catching the barrier as the middle beat. Drops the pre-render objective, which reveals nothing once the homepage is already clean. - Records that the vendor widget must be a same-document script, never an iframe. Focus and key events do not cross an iframe boundary, so a page-level beacon cannot observe the trap through one. Verified by spike. - Adds the schedule gap: a gate only ever checks the widget as of the last build, and the vendor can ship on a day when nobody pushes. - index.yaml overview now names all four layers, lint, scan, drive, observe, and what each one reaches. Signed-off-by: sindurigf <sinduri.g@gmail.com>
KatharinaSick
approved these changes
Sep 7, 2026
sindurigf
deleted the
idea/accessibility-nightmare-intermediate-expert
branch
September 7, 2026 09:06
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Closes #
Type of PR
💡 Adventure Idea
ideas/(notideas/.implemented/)🗺️ New Adventure Level
verify.shpasses when the challenge is solvedverify.shfails with helpful error messages when not solvedideas/.implemented/(first level PR only)Part of #for earlier levels,Closes #for the last one)📖 Solution Walkthrough
adventures/XX-.../docs/solutions/level.md)📝 Other