Implement mask-repeat: [space | round]

RESOLVED WORKSFORME

Status

()

defect
RESOLVED WORKSFORME
3 years ago
3 years ago

People

(Reporter: bmo, Assigned: ethlin)

Tracking

(Blocks 1 bug, {dev-doc-complete})

unspecified
mozilla49
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

3 years ago
The implementation will depend on bug548372.
(Reporter)

Updated

3 years ago
Duplicate of this bug: 1244609
Given that we don't support this for background-image, I don't think this needs to block shipping support for mask-image.

(It would be good to do it for both at some point, though.)
No longer blocks: mask-image, mask-ship
(Reporter)

Comment 4

3 years ago
(In reply to David Baron [:dbaron] ⌚️UTC-7 (review requests must explain patch) (busy May 9-13) from comment #3)
> Given that we don't support this for background-image, I don't think this
> needs to block shipping support for mask-image.
> 
> (It would be good to do it for both at some point, though.)

Agreed per your comments.
In fact, Ethan is working on bug 548372 and it's on the half way.
mask-repeat:space|round support can be done as soon as bug 548372 is fixed since they probably share most of the implementation. In that case, we are able to ship mask-image in one piece and it's the plan for now.
(Assignee)

Comment 5

3 years ago
I will work on this after bug 548372.
Assignee: nobody → ethlin
(Reporter)

Updated

3 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla49
(Assignee)

Comment 6

3 years ago
(In reply to Astley Chen [:astley] (UTC+8) from comment #1)
> Reference:
> [space] -
> http://astleychen.github.io/css-masking/bug1258623/mask-repeat-2-ref.html
> [round] -
> http://astleychen.github.io/css-masking/bug1258623/mask-repeat-3-ref.html
> 
> Tested:
> [space] -
> http://astleychen.github.io/css-masking/bug1258623/mask-repeat-2.html
> [round] -
> http://astleychen.github.io/css-masking/bug1258623/mask-repeat-3.html

After bug 548372, the above tests can pass on my device. I will send try server to make sure.
(Assignee)

Comment 7

3 years ago
try server result:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=3d3d5f8e8668&selectedJob=21507581

The tests 'mask-repeat-2.html' and 'mask-repeat-3.html' pass now.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WORKSFORME
The main page was already ok, so I just added a comment there:
https://developer.mozilla.org/en-US/Firefox/Releases/49#CSS

Updated

3 years ago
You need to log in before you can comment on or make changes to this bug.