`closing-remarks` - Prevent removal by GitHub by fregante · Pull Request #9498 · refined-github/refined-github · GitHub
Skip to content

closing-remarks - Prevent removal by GitHub #9498

Merged
fregante merged 4 commits into
mainfrom
closing-remarks-fo-sho
May 20, 2026
Merged

closing-remarks - Prevent removal by GitHub #9498
fregante merged 4 commits into
mainfrom
closing-remarks-fo-sho

Conversation

@fregante

@fregante fregante commented May 17, 2026

Copy link
Copy Markdown
Member

Test URLs

#9418

Screenshot

Video of me refreshing the page while holding cmd down to scroll to the bottom

Screen.Recording.mov

@fregante fregante added the bug label May 17, 2026

@fregante fregante May 17, 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.

This is the fix.

The code is awkward because it's backwards. Normally:

  • observe -> rest of the feature logic

This feature:

  • feature logic -> observe just to append

A full refactor would be beneficial, but closing-remarks is kinda on life support in the hope that GitHub implements it natively.

before: () => (
// Use observer because GitHub might remove the box
// https://github.com/refined-github/refined-github/issues/9460
observe(commentBoxHashPr, anchor => {

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.

This is for the "Not yet release, release now" banner. Not sure if it's affected too, but might as well just drop attachEvent here as well.

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.

Also tested and it still works:

Image

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 rest of the changes just clarify code a bit

This comment was marked as low quality.

@fregante fregante merged commit cf569f9 into main May 20, 2026
13 checks passed
@fregante fregante deleted the closing-remarks-fo-sho branch May 20, 2026 04:00
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.

closing-remarks removed by GitHub after being added

2 participants