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

Testing css #57

Open
djmtype opened this issue Dec 1, 2020 · 9 comments
Open

Testing css #57

djmtype opened this issue Dec 1, 2020 · 9 comments

Comments

@djmtype
Copy link

djmtype commented Dec 1, 2020

During development, npm run watch, how is one suppose to test css if the code has been minified? There are no line references because everything is on line 1.

@hhh968574

This comment has been minimized.

@cramforce
Copy link
Collaborator

The CSS should be unminified. There really is no good reason for the current state since it gets re-minified during build.

@djmtype
Copy link
Author

djmtype commented Dec 1, 2020

@cramforce True, the css isn't minified in the css file, but it does get transformed/minified (probably due to csso) during development – npm run watch. Therefore, there is no real frame of reference when troubleshooting.

Screen Shot 4

@cramforce
Copy link
Collaborator

Should be totally possible to generate a sourcemap!

@djmtype
Copy link
Author

djmtype commented Dec 1, 2020

@cramforce CSSO provides an option, but then the css gets parsed through PurgeCSS anyway, I think.

Please don't keep me in suspense. How can we add a css sourcemap for npm run watch only?

cramforce added a commit that referenced this issue Dec 12, 2020
Related to #57
@cramforce
Copy link
Collaborator

I filed FullHuman/purgecss#526

There is, indeed, little that can be done for now short of turning off CSS purging during development.

@theor
Copy link

theor commented Sep 28, 2021

I've disabled css inlining in dev mode, as it allows browsersync to inject css and bypass 11ty, which is way faster. is it something worth a PR ?

@cramforce
Copy link
Collaborator

I don't think I want to make that a default, but otherwise it is worth a PR for sure!

@theor
Copy link

theor commented Sep 30, 2021

#108 let me know what you think !

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

No branches or pull requests

4 participants