Skip to content

Fix process navigation, completion focus and authored colours - #1

Merged
willmot merged 2 commits into
mainfrom
codex/process-blocks-ux-fixes
Nov 3, 2026
Merged

willmot merged 2 commits into
mainfrom
codex/process-blocks-ux-fixes

Conversation

@willmot

@willmot willmot commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Checklist completion, Undo and reset could leave keyboard focus on a disappearing control; direct step links could land under the sticky bar, and an offscreen bar could identify the wrong action. Keep focus on a usable destination and measure navigation against the bar and viewport. Also deduplicate restored progress, let wide inner blocks shrink, and resolve authored completion colours through WordPress’s style engine and CSS filter, with separators compatible with WordPress 6.7.

Following Ryan’s fallback and generator feedback, hide progress controls until hydration while retaining readable steps and native summary links. Actions now run synchronously; only focus on a newly revealed control waits for the next frame.

Screenshots

The same existing coffee demo, with JavaScript disabled. Both captures are local WordPress previews.

Before this review follow-up (0069ae2) After (ef2a3b7)
Local preview before: inactive checklist controls and a JavaScript notice Local preview after: readable steps and summary links, with interactive controls hidden

Validation: npm run lint, npm run build and npm run plugin-zip passed; packaged files match the build. Local WordPress 6.7.11 and 7.1.4 checks passed in Chromium, Firefox and WebKit, covering focus, navigation, restored progress, colours, responsive layouts, multiple processes and script/storage failures. Local Playbook integration checks also passed. Physical Safari and assistive-technology testing remain unverified. CI passed for ef2a3b7.

- Written by Codex gpt-6.1-sol high

@willmot
willmot requested a review from rmccue October 30, 2026 09:14
Comment thread src/process/render.php Resolved
Comment thread src/process/view.js Resolved
@willmot
willmot requested a review from rmccue November 3, 2026 11:18

@rmccue rmccue left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

TIL there's a hidden attribute

@willmot
willmot merged commit c3ace02 into main Nov 3, 2026
2 checks passed
@willmot
willmot deleted the codex/process-blocks-ux-fixes branch November 3, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants