print rule name next to each warning (for easier suppression)

[mo]

Instead of printing just "Using width with border can sometimes make elements larger than you expect" it would be nice if it printed "box-model: Using width with border can sometimes make elements larger than you expect". This would make it easier to know what rule ID to pass to --ignore to suppress that particular error/warning.

This request for a thing that is similar to what -fdiagnostics-show-option does for C++ code in gcc.

[nschonni]

If you use a formatter other than compact I believe this is already the case. Ex: the text formatter https://github.com/stubbornella/csslint/blob/master/src/formatters/text.js#L56

[mo]

@nschonni, no it prints the rule text but not the id of rule. See my fix.

[nschonni]

I think it makes more sense in the text formatter though, compact is supposed to be small

[nschonni]

Or in compact it maybe swap the message for the rule name

[nzakas]

I think there's a larger question here of it the rule ID should be part of output at all. If it makes sense to have in compact output, then wouldn't you also expect it in other places?

[mo]

@nzakas, the fact that it's missing in all modes atm makes it very hard to know what param to pass when suppressing. I think it should be in both normal and compact modes but I only use compact mode so that's the one I fixed.

[nzakas]

To be clear: I believe this was an oversight and I would rather have this
oversight fixed in all appropriate places rather than make the change in
just one.

On Sat, Mar 9, 2013 at 12:58 PM, Martin Olsson notifications@github.comwrote:

@nzakas https://github.com/nzakas, the fact that it's missing in all
modes atm makes it very hard to know what param to pass when suppressing. I
think it should be in both normal and compact modes but I only use compact
mode so that's the one I fixed.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/355#issuecomment-14670516
.

---

Nicholas C. Zakas
@SlickNet

Author, Professional JavaScript for Web Developers
Buy it at Amazon.com:
http://www.amazon.com/Professional-JavaScript-Developers-Nicholas-Zakas/dp/1118026691/ref=sr_1_3

[epriestley]

#409 fixes it in one of the XML formatters. I can expand that to cover all the formatters if the lack of a comprehensive patch is the main thing blocking the pull.

[stubbornella]

@epriestley - if you are willing to make it work in all formatters, that would be great.