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 CSS Validator #451

Open
windymilla opened this issue Sep 27, 2024 · 2 comments
Open

Add CSS Validator #451

windymilla opened this issue Sep 27, 2024 · 2 comments
Labels
future feature New feature or request, but not core

Comments

@windymilla
Copy link
Collaborator

Is this necessary, or is it now covered by the Nu HTML checker?

@windymilla windymilla added the core feature Required for basic PPing label Sep 27, 2024
@tangledhelix
Copy link
Collaborator

I didn't test exhaustively, but I ran a few HTML files with intentionally-broken CSS through Nu, and it caught them. But Nu doesn't offer all the same options, as far as I can tell - you can't validate against CSS 2.1, for example, which is what we are supposed to do unless we know CSS3 will be okay (e.g., drop-caps). So it could be using CSS separately is the proper choice for now.

@tangledhelix
Copy link
Collaborator

Is this issue about bundling the Nu validator or a different validator with the app?

If so, IMO we should either not do that, and instead point users to the online validator, or at minimum move this to future feature New feature or request, but not core instead of core feature Required for basic PPing .

@windymilla windymilla added future feature New feature or request, but not core and removed core feature Required for basic PPing labels Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
future feature New feature or request, but not core
Projects
None yet
Development

No branches or pull requests

2 participants