Closed Bug 1562086 Opened 1 year ago Closed 1 year ago

[fill-stroke] [filter-effects] [css-color] percentage opacity

Categories

(Core :: CSS Parsing and Computation, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: mozilla-apprentice, Assigned: boris)

References

Details

(Keywords: dev-doc-complete, Whiteboard: [layout:backlog:2019q3][wptsync upstream])

Attachments

(1 file)

A resolution was made for csswg-drafts/#3342.

[fill-stroke] [filter-effects] [css-color] percentage opacity

  • RESOLVED: No change to spec

Discussion.

We should probably prioritize adding percentage opacity (in all the places it's allowed) relatively highly since once other engines add it, we're likely to start hitting compat problems reasonably quickly. It should also be pretty simple to do.

Component: SVG → CSS Parsing and Computation
Priority: P3 → P2
Assignee: nobody → boris.chiou

This bug will use <alpha-value> for these properties:

  1. opacity
  2. flood-opacity
  3. fill-opacity
  4. stroke-opacity
  5. stop-opacity
  6. -moz-window-opacity

-moz-window-opacity uses Opacity type, so for consistency, I will also update it.

Test: https://jsfiddle.net/ericwilligers/zmgy8735/

Type: -- → enhancement
Whiteboard: [layout:backlog:2019q3]

The following properties accept <alpha-value> [1], which is
"<number> | <percentage>", so we update the parser, spec links, and
their web-platform-tests.

  1. opacity
  2. flood-opacity
  3. fill-opacity
  4. stroke-opacity
  5. stop-opacity
  6. -moz-window-opacity

Besides, shape-image-threshold [2] still only accepts <number>, so we need
to support a different version of Opacity::parse().

[1] https://drafts.csswg.org/css-color/#typedef-alpha-value
[2] https://drafts.csswg.org/css-shapes/#shape-image-threshold-property

Keywords: dev-doc-needed
Pushed by bchiou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/abe58536c427
Add percentage for opacity (i.e. <alpha-value>). r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/17817 for changes under testing/web-platform/tests
Whiteboard: [layout:backlog:2019q3] → [layout:backlog:2019q3][wptsync upstream]
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Upstream PR merged
See Also: → 1568615
You need to log in before you can comment on or make changes to this bug.