Skip to content

open-all-conversations - Exclude issue views list - #9928

Merged
fregante merged 4 commits into
mainfrom
open-none
Aug 2, 2026
Merged

open-all-conversations - Exclude issue views list#9928
fregante merged 4 commits into
mainfrom
open-none

Conversation

@fregante

@fregante fregante commented Aug 2, 2026

Copy link
Copy Markdown
Member

@fregante fregante added the bug label Aug 2, 2026
function add(anchor: HTMLElement): void {
// TODO: Drop after https://github.com/refined-github/refined-github/issues/9893
if (closestElementOptional('[class*="RepositoryViews"]', anchor)) {
// The user navigate to https://github.com/refined-github/refined-github/issues/views but the previous observer was not unloaded

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.

Repro:

  1. open https://github.com/refined-github/refined-github/issues
  2. click Views in sidebar

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.

const selectedLinks = links.filter(link =>
closestElementOptional([
// TODO [2026-01-01]: Pre-React selector; Drop
// TODO [2027-01-01]: Drop if PR lists have turned React

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.

Way optimistic

}, {
include: [
pageDetect.isGlobalIssueOrPRList,
],

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.

We had this redundant logic for maaaaany years.

@fregante
fregante merged commit 89f6ca5 into main Aug 2, 2026
14 checks passed
@fregante
fregante deleted the open-none branch August 2, 2026 08:08
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.

open-all-conversations shows up on Views, but doesn't work

1 participant