Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 334920 - [content model] CSS3 missing border-bottom-left-radius property
Summary: [content model] CSS3 missing border-bottom-left-radius property
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.css (show other bugs)
Version: 3.2.3   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.2.3   Edit
Assignee: Nick Sandonato CLA
QA Contact: Nick Sandonato CLA
URL:
Whiteboard: PMC_approved
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-20 11:22 EST by Nick Sandonato CLA
Modified: 2011-01-27 13:05 EST (History)
2 users (show)

See Also:
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+

Attachments
patch (1.23 KB, patch)
2011-01-20 11:27 EST, Nick Sandonato CLA 		no flags Details | Diff
View All 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 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.