Skip to content

/ servo

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement repeating gradients (repeating-linear-gradient, repeating-radial-gradient) #10263

Closed
ocerman opened this issue Mar 29, 2016 · 5 comments
Closed

Implement repeating gradients (repeating-linear-gradient, repeating-radial-gradient) #10263

ocerman opened this issue Mar 29, 2016 · 5 comments
Labels
A-content/css C-assigned C-has-manual-testcase I-enhancement

Comments

@ocerman
Copy link

@ocerman ocerman commented Mar 29, 2016 •
edited by jdm

Tested on Windows 8.1 x64. Test html: http://testujem.eu/servo/repeating-gradient.html

<!DOCTYPE html>
<html>
<head>
    <meta charset="UTF-8">
    <title>Gradient test</title>
    <style>
        html{
            padding: 60px;
        }

        .gradient{
            height: 40px;;
            background: repeating-linear-gradient(45deg,darkblue 0,darkblue 22px,white 22px,white 44px,blue 44px,blue 66px,white 66px,white 88px);
        }
    </style>
</head>

<body>
    <div class="gradient"></div>
</body>

</html>
@mbrubeck mbrubeck changed the title Repeating gradient not working Implement repeating gradients (repeating-linear-gradient, repeating-radial-gradient) May 25, 2016
@mbrubeck
Copy link
Contributor

@mbrubeck mbrubeck commented May 25, 2016

https://drafts.csswg.org/css-images-3/#repeating-gradients
@dholbert dholbert mentioned this issue Jun 17, 2016
Closed
@pyfisch
Copy link
Contributor

@pyfisch pyfisch commented Apr 14, 2017

@highfive: assign me

This is a simple fix since webrender supports repeating gradients since a few days. The example works but I will wait until #16385 lands with a PR so I can add it for radial gradients too.
@highfive highfive added the C-assigned label Apr 14, 2017
@highfive
Copy link

@highfive highfive commented Apr 14, 2017

Hey @pyfisch! Thanks for your interest in working on this issue. It's now assigned to you!
@pyfisch
Copy link
Contributor

@pyfisch pyfisch commented Apr 29, 2017

@emilio solved by #16385. Please close. Thanks!
@jdm jdm closed this Apr 29, 2017
@jdm
Copy link
Member

@jdm jdm commented Apr 29, 2017

Thanks for fixing it!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-content/css C-assigned C-has-manual-testcase I-enhancement
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
@mbrubeck @jdm @pyfisch @highfive @ocerman
You can’t perform that action at this time.