Skip to content

batch-mark-files-as-viewed - Restore feature - #10231

Draft
fregante wants to merge 2 commits into
mainfrom
claude/batch-mark-files-as-viewed
Draft

fregante wants to merge 2 commits into
mainfrom
claude/batch-mark-files-as-viewed

Conversation

@fregante

@fregante fregante commented Oct 15, 2026 •

Copy link
Copy Markdown
Member

Created by Claude, will be reviewed by @fregante first.

GitHub dropped the *-module__ class prefixes. The toggle is now matched by its aria-label and the checked state by aria-pressed.

Alt-click also threw because markAsViewedSelector searched for the toggle inside the clicked toggle. It now resolves the file first.

Fixes #10229

Part of #10253

Test URLs:

Co-authored-by: Claude <noreply@anthropic.com>
@fregante

Copy link
Copy Markdown
Member Author

Shift-click works, but alt-click causes this error:

Expected element not found: button[aria-label="Viewed"],button[aria-label="Not Viewed"],input.js-reviewed-checkbox

Co-authored-by: Claude <noreply@anthropic.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

batch-mark-files-as-viewed - Selectors no longer match the PR Files view

1 participant