Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(735)

Issue 46080043: 'transparent' is now a CSS color (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 2 months ago by felix8a
Modified:
12 years, 2 months ago
Reviewers:
kpreid2
CC:
google-caja-discuss_googlegroups.com
Base URL:
http://google-caja.googlecode.com/svn/trunk/
Visibility:
Public.

Description

'transparent' used to be a special keyword in some CSS rules. CSS3 makes 'transparent' a color keyword, and it's valid anywhere a color is valid. This change makes 'transparent' a color.

Patch Set 1 #

Patch Set 2 : 'transparent' is now a CSS color #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -9 lines) Patch
M . View 1 0 chunks +0 lines, -0 lines 0 comments Download
M src/com/google/caja/lang/css/css3-defs.json View 1 5 chunks +9 lines, -6 lines 0 comments Download
M tests/com/google/caja/lang/css/CssPropertyPatternsTest.java View 1 1 chunk +1 line, -1 line 0 comments Download
M tests/com/google/caja/plugin/CssValidatorTest.java View 1 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 6
felix8a
12 years, 2 months ago (2013-12-27 22:18:37 UTC) #1
kpreid2
LGTM
12 years, 2 months ago (2013-12-27 23:08:06 UTC) #2
felix8a
'transparent' used to be a special keyword in some CSS rules. CSS3 makes 'transparent' a ...
12 years, 2 months ago (2013-12-28 00:17:37 UTC) #3
felix8a
updated snapshot for es53 branch. changing the css grammar affects some tests that expect particular ...
12 years, 2 months ago (2013-12-28 00:18:35 UTC) #4
kpreid2
LGTM
12 years, 2 months ago (2013-12-28 02:24:38 UTC) #5
felix8a
12 years, 2 months ago (2013-12-28 02:45:09 UTC) #6
@r5651 trunk
@r5653 branches/es53
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b