Code review comment for lp:~gero-bare/scratch/add-linter-support
Revision history for this message
| Jeremy Wootten (jeremywootten) wrote | # |
review:
Needs Fixing
(ui/ux)
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
On trying out the html linter I get two error messages, both for Line 0, Column 0:
1) Regex failure
2) Failed to execute child process "tidy" ....
until I installed the application "tidy". It would be better if an informative message or dialog was used to inform the user how to fix the problem.
Some work needs doing on the appearance of the configuration dialog in my opinion - needs UX team input.