Skip to content

conversation-activity-filter - Hide resolved review threads#9527

Merged
fregante merged 1 commit into
mainfrom
resolve-me-not
May 19, 2026
Merged

conversation-activity-filter - Hide resolved review threads#9527
fregante merged 1 commit into
mainfrom
resolve-me-not

Conversation

@fregante

Copy link
Copy Markdown
Member

@github-actions github-actions Bot added the bug label May 19, 2026
.js-resolvable-timeline-thread-container[data-resolved='true'],
.minimized-comment {
display: none;
display: none !important;

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The fix

}

function processDissmissedReviewEvent(item: HTMLElement): void {
function processDismissedReviewEvent(item: HTMLElement): void {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Typos and other comments

function applyState(targetState: State): void {
const container = $([
// Current PR view
// PR

@fregante fregante May 19, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

All code is for "current" views unless otherwise specified

@fregante

Copy link
Copy Markdown
Member Author

Just !important, merging

@fregante fregante merged commit e548c52 into main May 19, 2026
13 checks passed
@SunsetTechuila SunsetTechuila deleted the resolve-me-not branch May 21, 2026 10:58
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.

conversation-activity-filter doesn't hide resolved reviews

1 participant