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

Fixed issue where uploading would result in Show More being toggled incorrectly… #180

Merged
merged 2 commits into from Jan 7, 2023

Conversation

DaddyFrosty
Copy link
Contributor

@DaddyFrosty DaddyFrosty commented Jan 7, 2023

Fixed issue where uploading would result in Show More being toggled incorrectly…
Exact error: TypeError: Cannot read properties of undefined (reading 'click')

Reasoning is that Show More would be toggled twice resulting in elements not existing when they need to.

The fix is to ensure that the elements that are added as a result of pressing show more exist before proceeding with the code execution. In other words: Check if elements exist, if not press Show More, wait 1 second, check if everything exists. Proceed if everything is there.


This change is Reviewable

…ly pressed

Fixed issue where uploading would result in Show More being incorrectly pressed

Exact error: TypeError: Cannot read properties of undefined (reading 'click')

Reasoning is that Show More would be toggled twice resulting in elements not existing when they need to.
@DaddyFrosty
Copy link
Contributor Author

Please note that I did comment out the code on LN224 that toggles the button before adding playlists, there should be no reason for it to be called, but in the case that there is a reason for let me know and I can edit my commit.

@fawazahmed0
Copy link
Owner

Thanks

@fawazahmed0 fawazahmed0 merged commit d129f7f into fawazahmed0:main Jan 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants