Closed Bug 1382742 Opened 7 years ago Closed 7 years ago

stylo: site issue: Broken hover transition on fastaim.de

Categories

(Core :: CSS Parsing and Computation, defect, P3)

x86_64
Linux
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: jan, Unassigned)

References

(Blocks 1 open bug, )

Details

(Keywords: nightly-community)

Attachments

(2 files)

STR: 1. https://fastaim.de/test/ 2. Scroll down to the bottom 3. Hover from the left to the right over these buttons: Youtube, Twitter, Facebook, Steam 4. only with stylo enabled, the text+icon color sometimes remains white or pink. It can be fixed with hovering again the broken button (wait until the transition finished). Code seems to be okay: https://validator.w3.org/nu/?acceptlanguage=&doc=https%3A%2F%2Ffastaim.de%2Ftest%2F Off topic: I have read that xhtml (with a strong Content-Security-Policy) could be processed a bit more performant which could be good for mobile devices, can you confirm this?
This looks like the flickering issue bug 1380133 is fixing... > Off topic: I have read that xhtml (with a strong Content-Security-Policy) could be processed a bit more performant which could be good for mobile devices, can you confirm this? (No clue about where this comes from, nor whether it's true)
Priority: -- → P3
Yeah, I only see the flickering issue and confirmed that the patches for bug 1380133 fixed the flicker. I am going to make this as a duplicate of bug 1380133. Darkspirit, please feel free to reopen this if there is another problem in the site. Thank you for testing!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Doh! I've lost duplicate field during midair collision.
Resolution: FIXED → DUPLICATE
Attached video 2017-07-20_20-18-10.mp4 (deleted) —
Exceptionally I was on Windows now and only had a flickering there. Just to let you know what's happening in my main profile on Debian (comment 0) I uploaded a small video. I will retest when bug 1380133 landed.
It looks like bug 1381431 or bug 1371450 but they have been already fixed..
Attached video 2017-07-21_01-35-48.mp4 (deleted) —
Both videos are: Nightly 56 x64 20170720100139 @ Debian Testing (Linux 4.11.0-1-amd64, Radeon RX480). This one is with a fresh profile. I'm a bit desperate what differences could be the cause for different behavings. It's not hw acceleration. I have to dig deeper if bug 1380133 doesn't bring the solution.
Summary: icon font = fontawesome, http://fontawesome.io/icon/twitter/ for example The link text (icon font + text) is immediately white if I hover it. With servo disabled, there is a smooth transition from a red text color to a white one. In my main profile, both "icon font" and "regular text" color remain white sometimes. In a fresh profile, only the icon font color remains white sometimes. On Windows, there is some flickering, but not a smooth transition.
OK, I can confirm the white icon on debug build.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
According to Cameron's manual testing, looks like this can be fixed by bug 1380133.
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → DUPLICATE
Nightly 56 x64 20170724100304 @ Debian Testing (Linux 4.11.0-1-amd64, Radeon RX480). bug 1380133 fixed this partially. This bug still happens with visited links. I hope bug 1383307 will fix this in the next Nightly. (I will re-test.)
If the issue still remains for the visited links, I suspect this might be something related to bug 1381235.
Status: RESOLVED → REOPENED
Depends on: 1380133, 1381235
Resolution: DUPLICATE → ---
I can confirm that my patch in bug 1381235 can fix the remaining issues in this bug. \o/
Summary: stylo: site issue: Broken hover transition → stylo: site issue: Broken hover transition on fastaim.de
This is fixed in the "Linux x64 opt" try build of bug 1381235 comment 14. I tested both unvisited an visited links in a fresh profile with Stylo enabled. All the bugs are dying :)
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → FIXED
Verified fixed in Nightly 56 x64 20170729100254 @ Debian Testing. Thank you!
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: