Skip to content
New issue

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

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

Already on GitHub? # to your account

Add an error message when a js file failed to load #53

Merged
merged 1 commit into from
Nov 27, 2019

Conversation

uiryuu
Copy link
Contributor

@uiryuu uiryuu commented Nov 24, 2019

Show an error message when a JS file failed to load.

I have to add some dummy value to the disabled entries, otherwise, they might be pre-selected before you open the dropdown menu. It seems like a react-semantic-ui bug. See Semantic-Org/Semantic-UI-React#3130 (comment).

Any wording improvement welcomed.

Fix #46.

Test plan:
My setup is Chrome + uBlock Origin with all default filters + https://www.test.com.
You should see a JavaScrtipt file is blocked.

@uiryuu uiryuu added the enhancement New feature or request label Nov 24, 2019
@uiryuu uiryuu requested a review from a team November 24, 2019 23:07
Copy link
Contributor

@williamtdr williamtdr left a comment

Choose a reason for hiding this comment

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

Perfect! Nice work.

@rafibarash
Copy link
Collaborator

yo this looks great

@rafibarash rafibarash merged commit 142f125 into master Nov 27, 2019
@rafibarash rafibarash deleted the error-component branch November 27, 2019 16:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error component if a JS artifact fails to load
3 participants