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

deploy/gh-pages: tidy output before deploy #10

Open
sidvishnoi opened this issue Aug 26, 2020 · 4 comments
Open

deploy/gh-pages: tidy output before deploy #10

sidvishnoi opened this issue Aug 26, 2020 · 4 comments
Labels
enhancement Enhancement to an existing feature help wanted Extra attention is needed

Comments

@sidvishnoi
Copy link
Member

sidvishnoi commented Aug 26, 2020

So, the diff is easier to read, if any. Also, deployed output is easier to read with view-source
Will also play nice with #8

@sidvishnoi sidvishnoi added the enhancement Enhancement to an existing feature label Aug 26, 2020
@sidvishnoi sidvishnoi added the help wanted Extra attention is needed label Feb 5, 2021
@sidvishnoi
Copy link
Member Author

Help wanted: use prettier or html5-tidy?

Prettier: Easy to install and run. Creates awkward output many times (self-closing tags with trailing slash, tags with many attributes go multi-line awkwardly).
Tidy: A bit more difficult to install, updated less frequently. Formatting feels more natural.

@sidvishnoi
Copy link
Member Author

Prettier with --html-whitespace-sensitivity=ignore seems nice, though it can have negative effect on layout and crawlers like reffy and specberus.

@marcoscaceres
Copy link
Member

Maybe we should make this configurable (and default to html5 tidy).

@marcoscaceres
Copy link
Member

Ran into wanting this today.... just just bumping for myself.

Maybe we can do another coding session for this in the coming month after TPAC?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement to an existing feature help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants