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

Regression in the rendering of rounded borders #2754

Closed
Zirro opened this issue May 11, 2018 · 12 comments
Closed

Regression in the rendering of rounded borders #2754

Zirro opened this issue May 11, 2018 · 12 comments

Comments

@Zirro
Copy link

@Zirro Zirro commented May 11, 2018

There seems to have been a recent regression in the way rounded borders are rendered. Here is a test case, with screenshots below.

Without WebRender:

Without WebRender

With WebRender:

With WebRender

This seems to have changed within the last month, as the circle can be seen rendering properly in this issue created at the end of March.

@Darkspirit
Copy link

@Darkspirit Darkspirit commented May 14, 2018

I couldn't reproduce it to look like your screenshot, but I filed a bug based on your testcase: https://bugzilla.mozilla.org/show_bug.cgi?id=1461513

Edit: I think this test case is just a basic (and easier to reproduce) variant of #2590.

@Zirro
Copy link
Author

@Zirro Zirro commented May 15, 2018

@Darkspirit This could be the case, as the flickering boxes (which now appear if I make the window small enough to scroll) seem to align with some of the missing parts of the border.

That said, something must still have changed as I used to see the dots in the circle on https://rawgit.com/Zirro/03f0e2d19b087db29b1f069e014e3ee5/raw/0f32535d70e56185e01e4f9be860afc4dc4fec79/fireworks.html when I created that issue, and for some time afterwards.

I will have another look at this test case when #2590 has been resolved, to see if they are indeed the same issue or two separate ones.

@mrobinson
Copy link
Member

@mrobinson mrobinson commented May 25, 2018

Hrm. I'm having trouble reproducing this one as well. I do see artifacts in https://rawgit.com/Zirro/03f0e2d19b087db29b1f069e014e3ee5/raw/0f32535d70e56185e01e4f9be860afc4dc4fec79/fireworks.html, though they don't seem to be related to to dot or dash rendering.

@mrobinson
Copy link
Member

@mrobinson mrobinson commented May 25, 2018

Okay. I do see artifacts on this test case when resizing the window. These artifacts seem to affect the borders and the backgrounds equally though -- which are two different code paths.

@Zirro
Copy link
Author

@Zirro Zirro commented May 27, 2018

If it can help to narrow down the specific rendering issues I'm seeing by looking at certain hardware/drivers, I'm on a MacBook Pro (Early 2013) with Intel HD Graphics 4000 and a NVIDIA GeForce GT 650M with High Sierra.

@kvark
Copy link
Member

@kvark kvark commented May 30, 2018

Works fine on my end (Linux/Intel) with or without resizing.

@MartinRogalla
Copy link

@MartinRogalla MartinRogalla commented Jun 7, 2018

Seeing the same issue on MacBookPro12,1 (Retina, 13-inch, Early 2015) w/ Intel Iris Graphics 6100.

screen shot 2018-06-07 at 17 46 50

@Zirro
Copy link
Author

@Zirro Zirro commented Jun 12, 2018

This appears to have been resolved - quite likely through the merging of #2795. This is what I see now:

Screenshot

The border placement could use some improvements to match the non-WebRender version, but I think this issue can be closed at this point.

Latest WebRender has not been merged into Servo yet, but is available in Firefox Nightly.
@MartinRogalla Could you verify whether this is fixed in latest Nightly for you?

@Darkspirit
Copy link

@Darkspirit Darkspirit commented Jun 12, 2018

Better for me too. The dashed border needs some love and the bottom-right dot of the top-left box looks bitten off.

@MartinRogalla
Copy link

@MartinRogalla MartinRogalla commented Jun 12, 2018

Looks a bit better, but I see some strange behaviour on Nightly. The elements change if I click on them. I'm not sure if this is related. Please see the attached GIF:
out

Please let me know if you want me to provide any additional information.

@Zirro
Copy link
Author

@Zirro Zirro commented Jun 12, 2018

That is strange behaviour indeed. For what it's worth, I can't seem to reproduce it on my machine.

@gw3583
Copy link
Collaborator

@gw3583 gw3583 commented Jun 18, 2018

It seems like the original bug has been fixed. We should probably close this bug and open a new one with the behavior mentioned above by @MartinRogalla - I'm also unable to reproduce that issue on any of my local test machines - it may be a driver specific issue.

bors-servo added a commit that referenced this issue Jul 4, 2018
Work around OSX/Intel shader bug with integer comparison

Fixes #2862
Fixes #2754

Oh OSX GL drivers, so rough, such cruel, no remorse.
r? anyone

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2864)
<!-- Reviewable:end -->
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
6 participants
You can’t perform that action at this time.