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

Bug 334920

Summary: [content model] CSS3 missing border-bottom-left-radius property
Product: [WebTools] WTP Source Editing Reporter: Nick Sandonato <nsand.dev>
Component: wst.cssAssignee: Nick Sandonato <nsand.dev>
Status: RESOLVED FIXED QA Contact: Nick Sandonato <nsand.dev>
Severity: normal    
Priority: P3 CC: raghunathan.srinivasan, thatnitind
Version: 3.2.3Flags: nsand.dev: pmc_approved? (david_williams)
raghunathan.srinivasan: pmc_approved+
nsand.dev: pmc_approved? (naci.dai)
nsand.dev: pmc_approved? (deboer)
nsand.dev: pmc_approved? (neil.hauge)
nsand.dev: pmc_approved? (kaloyan)
thatnitind: review+
Target Milestone: 3.2.3   
Hardware: PC   
OS: Windows XP   
Whiteboard: PMC_approved
Attachments:
Description Flags
patch none

Description Nick Sandonato CLA 2011-01-20 11:22:41 EST
It looks like we're missing the new border-bottom-left-radius property in the CSS3 profile.
Comment 1 Nick Sandonato CLA 2011-01-20 11:27:44 EST
Created attachment 187208 [details]
patch

Fixes a copy/paste mistake.
Comment 2 Nick Sandonato CLA 2011-01-21 10:53:44 EST
* Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such.

Since this is our first release with CSS3 support, we want to make sure it is as correct and as complete as possible. 

* Is there a work-around? If so, why do you believe the work-around is insufficient?

No.

* How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added?

Ad hoc testing.

* Give a brief technical overview. Who has reviewed this fix?

The problem was a copy/paste mistake, so the content model was never made aware of boorder-bottom-left-radius being a CSS3 property, but instead border-bottom-right-radius was read in as a property twice.

Nitin has reviewed the patch.

* What is the risk associated with this fix?

None, it was a copy/paste mistake.
Comment 3 Nick Sandonato CLA 2011-01-21 13:58:10 EST
Thanks for the quick review. Code has been released.