Skip to content

toggle-everything-with-alt - Support "Resolve conversation"#9310

Merged
fregante merged 4 commits into
refined-github:mainfrom
mvanhorn:fix/toggle-everything-with-alt-resolve-conversation
Apr 30, 2026
Merged

toggle-everything-with-alt - Support "Resolve conversation"#9310
fregante merged 4 commits into
refined-github:mainfrom
mvanhorn:fix/toggle-everything-with-alt-resolve-conversation

Conversation

@mvanhorn

Copy link
Copy Markdown

Closes #9309.

Alt-clicking a "Resolve conversation" or "Unresolve conversation" button in a PR conversation now applies the same action to every other thread in the same state, matching the rest of the feature's behavior on collapse togglers, suggestion-batching, etc.

The selector keys off the form action (/resolve vs /unresolve) rather than the button label so it doesn't depend on visible text or i18n. The clicked button's parent form is used to decide which set to fan out to: alt-clicking Resolve resolves every other unresolved thread; alt-clicking Unresolve unresolves every other resolved thread.

Test URLs

  • PR with multiple unresolved threads (alt-click "Resolve conversation"): PR with many reviews yolo#8
  • Same PR after resolving (alt-click "Unresolve conversation" on a resolved thread)

Screenshot

(Behavior is "click happens on every matching thread"; same fan-out as other handlers in this feature, no new visual surface.)

🤖 Authored with Claude Code

Alt-clicking a "Resolve conversation" or "Unresolve conversation"
button in a PR conversation now applies the same action to every
other thread in the same state, matching the rest of the feature's
behavior on collapse togglers, suggestion-batching, etc.

The selector keys off the form action (`/resolve` vs `/unresolve`)
rather than the button label so it doesn't depend on visible text
or i18n.

Closes refined-github#9309
@github-actions github-actions Bot changed the title toggle-everything-with-alt: support "Resolve conversation" toggle-everything-with-alt: support "Resolve conversation" Apr 30, 2026
@fregante

Copy link
Copy Markdown
Member

(Behavior is "click happens on every matching thread"; same fan-out as other handlers in this feature, no new visual surface.)

Can you include a demo video or gif? Untested AI PRs are not merged, the demo will tell us that you actually tested it

@fregante fregante changed the title toggle-everything-with-alt: support "Resolve conversation" toggle-everything-with-alt - Support "Resolve conversation" Apr 30, 2026
@SunsetTechuila

SunsetTechuila commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Note: The form is added to the DOM only after a hidden conversation is expanded. If you've just opened the page and want to bulk-unresolve comments, you won't be able to

@fregante

Copy link
Copy Markdown
Member

Yeah this is kind of an issue with all the selections, they're all "best effort". For example batch-mark-as-viewed is further degraded by React removing files outside of the viewport from the dom.

@SunsetTechuila

Copy link
Copy Markdown
Contributor

React removing files outside of the viewport from the dom

It's TanStack Virtual 🤓

@fregante

Copy link
Copy Markdown
Member

Tested

rg

@fregante fregante enabled auto-merge (squash) April 30, 2026 18:25
@fregante fregante merged commit ed3f025 into refined-github:main Apr 30, 2026
9 checks passed
@fregante

Copy link
Copy Markdown
Member

@mvanhorn 🙏

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

Development

Successfully merging this pull request may close these issues.

toggle-all-things-with-alt support "Resolve conversation" in PRs

3 participants