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

Add failing CSS part to HMR demo #31

Merged
merged 6 commits into from
Mar 15, 2022

Conversation

fregante
Copy link
Contributor

@fregante fregante commented Feb 2, 2022

Follows #29

It seems that the error manifests even with the basic CSS extraction configuration, but only in serve mode; webpack build works correctly.

This might not be a bug of this plugin

@Jack-Works
Copy link
Member

Ok, so I'll try to fix the problem with the CSS extractor/loader. But maybe it's not a bug within this project and I may not start the work right now. Maybe a few days I'll come back.

@Jack-Works
Copy link
Member

😭 Sorry for the delay but I still don't have time to investigate this problem. I have a deadline today! But I promise this issue is on my TODO list for a long time

@fregante
Copy link
Contributor Author

No worries, same here, I haven't had time to test your examples PR either 😅

@Jack-Works
Copy link
Member

well, it works, but by a very unwanted patch.

@Jack-Works
Copy link
Member

It's very hacky, and the example also need rewrite

@Jack-Works Jack-Works requested a review from crimx March 13, 2022 05:32
@Jack-Works
Copy link
Member

cc @fregante @crimx plz review this

README.md Show resolved Hide resolved
@Jack-Works Jack-Works requested a review from crimx March 13, 2022 14:04
@crimx crimx merged commit f3a2352 into awesome-webextension:master Mar 15, 2022
@fregante fregante deleted the failing-css-hmr branch March 23, 2022 04:44
@fregante
Copy link
Contributor Author

fregante commented Mar 23, 2022

Tested, it works! Thank you for the help

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.

3 participants