Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 343299 - CSS formatter breaks IE filter property values
Summary: CSS formatter breaks IE filter property values
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.css (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.3 M7   Edit
Assignee: Nick Sandonato CLA
QA Contact: Nick Sandonato CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-19 14:12 EDT by Ian Brandt CLA
Modified: 2011-04-25 16:29 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Brandt CLA 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 CLA 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 CLA 2011-04-19 15:28:06 EDT 
Changes to the formatter checked into HEAD.
Comment 3 Ian Brandt CLA 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 CLA 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 CLA 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.