Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 343299

Summary:	CSS formatter breaks IE filter property values
Product:	[WebTools] WTP Source Editing	Reporter:	Ian Brandt <ian>
Component:	wst.css	Assignee:	Nick Sandonato <nsand.dev>
Status:	RESOLVED FIXED	QA Contact:	Nick Sandonato <nsand.dev>
Severity:	normal
Priority:	P3	CC:	ian
Version:	unspecified
Target Milestone:	3.3 M7
Hardware:	All
OS:	All
Whiteboard:

Description Ian Brandt

2011-04-19 14:12:31 EDT

Build Identifier: 20110218-0911

When formatting the CSS Editor reformats Microsoft "filter" property ( http://msdn.microsoft.com/en-us/library/ms532847(v=vs.85).aspx ) values by introducing spaces in such a way that breaks IE's ability to recognize them.

Reproducible: Always

Steps to Reproduce:
1. Preferences... > Web > CSS Files > Editor > Line width: 120
2. Open a CSS file with the following rule:
.gradient {
	filter: progid:DXImageTransform.Microsoft.gradient(startColorstr='#FFF', endColorstr='#000');
}
3. Source > Format

The editor reformats the above to:

.gradient {
	filter: progid : DXImageTransform.Microsoft.gradient ( startColorstr = '#F0F1F3', endColorstr = '#DCDDE2' );
}

The spaces introduced between "progid : DXImageTransform" cause IE to no longer recognize the property value.

Comment 1 Ian Brandt

2011-04-19 14:15:01 EDT

Note that there should be no line wraps in the CSS properties above, Bugzilla wrapped them.

Comment 2 Nick Sandonato

2011-04-19 15:28:06 EDT

Changes to the formatter checked into HEAD.

Comment 3 Ian Brandt

2011-04-25 14:21:31 EDT

(In reply to comment #2)
> Changes to the formatter checked into HEAD.

Thanks for the quick fix!  If I'm understanding the release process it looks like my best bet to verify would be to watch for the next EPP JEE nightly package build (http://www.eclipse.org/epp/download.php) for my platform?  Barring that it appears that the Indigo M7 packages are due out a week from this coming Friday (http://wiki.eclipse.org/Indigo/Simultaneous_Release_Plan#Milestones_and_Release_Candidates).  Let me know if I wouldn

Comment 4 Ian Brandt

2011-04-25 14:23:23 EDT

(In reply to comment #3)
>  Let me know if I wouldn

[Wrong key]

...wouldn't find this commit in either of those builds.

Comment 5 Nick Sandonato

2011-04-25 16:29:34 EDT

(In reply to comment #4)
> (In reply to comment #3)
> >  Let me know if I wouldn
> 
> [Wrong key]
> 
> ...wouldn't find this commit in either of those builds.

Yup. Just keep an eye out for either of these.