Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Console Errors #2290

Closed
yakov116 opened this issue Jul 31, 2019 · 6 comments · Fixed by #2292
Closed

Console Errors #2290

yakov116 opened this issue Jul 31, 2019 · 6 comments · Fixed by #2292
Labels

Comments

@yakov116
Copy link
Member

❌ edit-readme
content.js:2154 TypeError: Cannot read property 'dataset' of null
    at edit_readme_init (content.js:6917)
    at features_run (content.js:2149)
    at content.js:2195
    at content.js:2110
    at onAjaxedPagesRaw (content.js:2105)

On any page

content.js:2082 Uncaught (in promise) TypeError: Failed to execute 'observe' on 'MutationObserver': parameter 1 is not of type 'Node'.
    at onNewsfeedLoad (content.js:2082)
@yakov116 yakov116 added the bug label Jul 31, 2019
@fregante
Copy link
Member

@yakov116
Copy link
Member Author

never mind looks like it was an error on my end

@notlmn
Copy link
Contributor

notlmn commented Jul 31, 2019

  • can't reproduce the second one. That code only runs on isDashboard

Can reproduce on issues and PRs on forced reload. That's not related to user dashboard, this issue looks like it's from show-names. Specifically onNewFeedLoad function used as load function for one of show-names's declaration in #2229.

image

This function should not be called on issues/PRs, but somehow it is and fails on https://github.com/sindresorhus/refined-github/blob/bc7d35cda229ea5e282f8cb6f5a62b9e09c1f5a0/source/libs/on-newsfeed-load.ts#L11. Causing elementReady on https://github.com/sindresorhus/refined-github/blob/bc7d35cda229ea5e282f8cb6f5a62b9e09c1f5a0/source/libs/on-newsfeed-load.ts#L17 to go into a rAF callback forever. Was experiencing something similar when profiling #2087 (the little callbacks in this screenshot on the either side).

image

I suspect this might be the same issue as #2273.

@yakov116 yakov116 reopened this Jul 31, 2019
@yakov116
Copy link
Member Author

@fregante same as above

@fregante
Copy link
Member

Try #2292

@notlmn those rAFs aren't ours; elementReady stops on dom ready.

@notlmn
Copy link
Contributor

notlmn commented Aug 1, 2019

@notlmn those rAFs aren't ours; elementReady stops on dom ready.

Sorry, found that those are from hide-files-on-github: sindresorhus/hide-files-on-github#82.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

3 participants