X Tutup
Skip to content

clean-conversation-headers - Fix issues cased by history navigation#9004

Merged
SunsetTechuila merged 3 commits intomainfrom
clean-pr-header-history
Feb 24, 2026
Merged

clean-conversation-headers - Fix issues cased by history navigation#9004
SunsetTechuila merged 3 commits intomainfrom
clean-pr-header-history

Conversation

@SunsetTechuila
Copy link
Member

@SunsetTechuila SunsetTechuila commented Feb 22, 2026 •
edited
Loading

Fixes #9001

All of these selectors target the summary row, which always contains three links: one to the author/person who merged the PR (always first in the DOM), and two to branches:

refined-github/source/features/clean-conversation-headers.tsx

Lines 85 to 91 in 88883bc

observe([
'span[class*="PullRequestHeaderSummary"]',
// Old views. TODO: Remove after July 2026
'.gh-header-meta > .flex-auto', // Real
'.rgh-conversation-activity-filter', // Helper in case it runs first and breaks the `>` selector, because it wraps the .flex-auto element
'[class^="StateLabel"] + div > span:first-child',
], cleanPrHeader, {signal});

Alternative solution would be to use elementReady

Test URLs

https://github.com/refined-github/refined-github/pulls

Screenshot

msedge_iNnYFMvNhv

@github-actions github-actions bot added the bug label Feb 22, 2026
@SunsetTechuila SunsetTechuila merged commit 5c63f32 into main Feb 24, 2026
8 checks passed
@SunsetTechuila SunsetTechuila deleted the clean-pr-header-history branch February 24, 2026 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fregante fregante fregante approved these changes

Assignees

No one assigned

Labels

bug

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

clean-conversation-headers fails on history navigation

2 participants

@SunsetTechuila @fregante
X Tutup