Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix deprioritize-marketplace-link error in some instances - #3919

Merged
fregante merged 4 commits into
refined-github:mainfrom
cheap-glitch:fix-on-profile-dropdown-load
Jan 31, 2021
Merged

fregante merged 4 commits into
refined-github:mainfrom
cheap-glitch:fix-on-profile-dropdown-load

Conversation

@cheap-glitch

Copy link
Copy Markdown
Contributor

Thanks for contributing! 馃崉

  1. LINKED ISSUES: Fixes deprioritize-marketplace-link: Argument 1 ('target') to MutationObserver.observe must be an instance of Node聽#3898

  2. TEST URLS: Go to the PR list and do a few "hard refresh" (Ctrl-F5). This triggers the error almost every time for me.

I think #3898 was caused by awaitDomReady: false combined with the plain document.querySelector() in onProfileDropdownLoad(). Using elementReady seems to fix it.

Also set-default-repositories-type-to-sources is unaffected.

Comment thread source/github-events/on-profile-dropdown-load.ts Outdated
@yakov116 yakov116 added the bug label Jan 28, 2021

@fregante fregante left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM if it works

@fregante fregante changed the title Fix onProfileDropdownLoad() Fix deprioritize-marketplace-link error in some instances Jan 30, 2021
@fregante
fregante merged commit cfd47cb into refined-github:main Jan 31, 2021
@fregante

Copy link
Copy Markdown
Member

woot woot for bugfixes!

@cheap-glitch
cheap-glitch deleted the fix-on-profile-dropdown-load branch January 31, 2021 08:53
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.

deprioritize-marketplace-link: Argument 1 ('target') to MutationObserver.observe must be an instance of Node

3 participants