vtst / ow

Various Eclipse plugins for web development
http://www.normalesup.org/~simonet/soft/
74 stars 29 forks source link

LESS: Validation settings feature request, and a minor bug report. #178

Closed coredumperror closed 10 years ago

coredumperror commented 10 years ago

First, let me say that I love the new features that you've added since I last checked out this plugin. I can finally toss out Aptana Studio, now that I no longer need its CSS editor to get halfway sane LESS syntax coloring.

There's only one significant problem I can think of: it's too hard to find the settings that let you disable validation (error reporting). The fact that those settings are only applicable at the project level is too restrictive. It should be possible to apply them at the Workspace level (i.e. directly from the Eclipse preferences dialog). This will make it easier to find the settings (I wasn't even aware that there were any Validation settings until I dug around in the project properties in desperation), and less annoying for folks who have lots of projects.

The reason that I find the need to disable error reporting is due to issue #147. Every project my shop has uses LESS in the way described in that issue, which makes this plugin incorrectly report hundreds of errors throughout almost every LESS file we've got.

In addition, even with every project-level Validation setting disabled, there is still one type of warning that appears in my LESS editors: "Applied parameters do not match any definition of this mixin." This is a fairly minor bug, since it's only a warning, but it'd be nice to see it fixed.

vtst commented 10 years ago

The warning can now be disabled. For workspace-level preference, see #141.

coredumperror commented 10 years ago

Excellent, thank you!