WeBuilder 2008 Remove comments in CSS Formatter adds them

[garethjmsaunders]

I've just been using the CSS Formatter in WeBuilder 2008 (9.0.0.96) and have discovered that the "Remove comments" option under the Misc tab actually includes comments rather than removes them. This appears to me to be a bug.

Here's what I do:

When I go to CSS > Format CSS Code. The settings I currently have are:

Rules tab: Rule format: Single line
Selectors tab: Selector order: Unchanged / Selector case: Unchanged
Properties tab: Property case: Unchanged / Property order: Ascending (alphabetic)
Shorthand Properties tab: <nothing selected>
Misc tab: Indent size: 2 / Indent with: Space / Color format: Unchanged

Based on these settings the Preview shows this:

body {color: #808080; font-family: Arial, Helvetica, sans-serif; font-size: 10pt;}
a {color: #808000;}
a:visited {color: #808000;}
#main {margin-bottom: 3px; margin-left: 2px; margin-right: 2px; margin-top: 3px;}

However, if I then select "Remove comments" it shows this:

/* Sample CSS Code */body {color: #808080; font-family: Arial, Helvetica, sans-serif; font-size: 10pt;}
a {color: #808000;}
a:visited {color: #808000;}
#main {margin-bottom: 3px; margin-left: 2px; margin-right: 2px; margin-top: 3px;}

In other words, it ADDS rather than removes CSS comments.

When applying these settings to an actual CSS file I can confirm that it retained all the CSS comments. It's almost as though this "switch" is wired back-to-front. Or the option is labelled incorrectly and should read "Keep comments" or "Include comments" or similar.

I hope that's a help.

Gareth

[Karlis]

You seem to be right.

[Mot]

More concisely stated, the CSS Formatter "Remove Comments" toggle is backwards.

An easy correction for a decimal point release - and to confuse the crap out of us again when it starts working right and we have it set the other way. :p