Skip to content
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

Bug on spinning.html with webrender #11759

Closed
negge opened this issue Jun 16, 2016 · 6 comments
Closed

Bug on spinning.html with webrender #11759

negge opened this issue Jun 16, 2016 · 6 comments
Labels

Comments

@negge
Copy link
Contributor

@negge negge commented Jun 16, 2016

Using master servo with -w (webrender), the spinning.html drops some of the rectangles. This can be seen with the commands:

without webrender:
./mach run -r -- https://people.xiph.org/~negge/spinning.html
https://people.xiph.org/~negge/servo_spinning_nowr.png

with webrender:
./mach run -r -w -- https://people.xiph.org/~negge/spinning.html
https://people.xiph.org/~negge/servo_spinning_wr_bug.png

@nox nox changed the title Bug on spinning.html with webrenderer Bug on spinning.html with webrender Jun 20, 2016
@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Jun 30, 2016

I saw this on some Android devices, too. Seemed to be tied to the border radius, as the problem went away on that device (Samsung Galaxy Tab 2) if I removed it:
https://github.com/larsbergstrom/webrender-demos/blob/use_raf/transparent_rects.html#L82

@cbrewster
Copy link
Member

@cbrewster cbrewster commented Jun 30, 2016

I set the rotation to a static -24deg and while resizing the window, it looks like the ones on the edge are being clipped prior to the rotation happening.
spinning_clip_issue

@cbrewster
Copy link
Member

@cbrewster cbrewster commented Jun 30, 2016

Adding P1 as I assume we want all the linked demos on the newtab page to work properly. Feel free to change it if it shouldn't be P1.

@jdm
Copy link
Member

@jdm jdm commented Jun 30, 2016

This doesn't feel like it impacts the demo in a meaningful way, so I'm dropping the "omg terrible user experience must fix" label :)

@cbrewster
Copy link
Member

@cbrewster cbrewster commented Jun 30, 2016

Better gif of the original issue:
spinning_issue 6 49 05 pm

@nox
Copy link
Member

@nox nox commented Oct 2, 2017

Closing as duplicate of #11970.

@nox nox closed this Oct 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.