floating text not work in www.bilibili.tv (assumes that if -moz-transition works then -moz-transform works)
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr115 | --- | unaffected |
firefox119 | --- | unaffected |
firefox120 | --- | disabled |
firefox121 | --- | disabled |
firefox122 | --- | fixed |
People
(Reporter: mix5003, Assigned: emilio)
References
(Depends on 1 open bug, Regressed 1 open bug, Regression)
Details
(Keywords: regression)
Attachments
(2 files)
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:121.0) Gecko/20100101 Firefox/121.0
Steps to reproduce:
- go to some video in bilibili.tv that has floating text (danmaku feature) for example https://www.bilibili.tv/en/play/2090295/12975589
- enable floating text (danmaku feature)
- wait for floating text appear
Actual results:
floating text not visible when video play. but if it pause it can see normally
Expected results:
floating text should visible when video playing.
like in my video. first part from chrome it can visible floating text normally. but second part is firefox i can't see any text until i pause video
info form mozregression
2023-10-28T00:17:32.706000: DEBUG : Found commit message:
Bug 1855763 - Disable -moz-transform and enable zoom on Nightly. r=zcorpan
Disable zoom on a couple tests that otherwise fail. They're known
issues:
- offsetWidth/Height not being unzoomed (button-min-width).
- getComputedStyle not being unzoomed (font-size-interpolation-003.html)
- ResizeObserver box not being unzoomed (https://github.com/w3c/csswg-drafts/issues/9398).
Differential Revision: https://phabricator.services.mozilla.com/D189644
![]() |
||
Updated•8 months ago
|
Comment 2•8 months ago
|
||
The Bugbug bot thinks this bug should belong to the 'Core::Audio/Video: Playback' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.
Comment 3•8 months ago
|
||
:emilio, since you are the author of the regressor, bug 1855763, could you take a look? Also, could you set the severity field?
For more information, please visit BugBot documentation.
Assignee | ||
Comment 4•8 months ago
|
||
I can't load the video attached to comment 0, and the link there says "Sorry, according to the request of the copyright owner, this video is not available at the moment.".
Any chance you could provide another URL that shows the issue? Thanks.
can you try this link (but i think flying text base on region too, so it may not appear in your region)
https://www.bilibili.tv/th/video/4789014123387392
https://www.bilibili.tv/th/video/4789014009878016
or can you pick randomly series in this page https://www.bilibili.tv/en/anime
i think i can create reproduction case: https://jsfiddle.net/ysdx2fkg/2/
i think key point is use transition: -moz-transform
and normal transform
css. i am not sure is this your intention or not
and when i change layout.css.prefixes.transforms
to true. it fixed this bug.
Assignee | ||
Comment 7•8 months ago
|
||
Okay, so here's the relevant code:
r.prototype.renderStyle=function(e,i,s){for(var l="",p=0;p<g.browserPrefix.length;p++)l+=g.browserPrefix[p]+"transform: translateX("+(e?1:-1)*i+`px) translateY(0px) translateZ(0px);
`+g.browserPrefix[p]+"transition: "+g.browserPrefix[p]+"transform "+s+"s linear;";this.element.style.cssText=this.cssTest+l}
Where d.browserPrefix=["","-webkit-","-moz-","-ms-"];
. So they're assuming that if -moz-transition
works, then -moz-transform
works too.
I guess let's try to get rid of the transition prefixes too...
Assignee | ||
Comment 8•8 months ago
|
||
Can you confirm that changing layout.css.prefixes.transforms
back to false, and flipping layout.css.prefixes.transitions
too, then it works again?
yes if i set layout.css.prefixes.transforms
and layout.css.prefixes.transitions
to false. it can fix this bug.
Assignee | ||
Updated•8 months ago
|
Comment 11•8 months ago
|
||
Set release status flags based on info from the regressing bug 1855763
Assignee | ||
Comment 12•8 months ago
|
||
Code like the one from comment 7 seems reasonable. Let's try to disable
prefixed transitions along with prefixed transforms in order to minimize
the compat fallout from CSS zoom.
While I was at it I also fixed the prefixed animation entries in the
property database and crashtests. But those aliases remain enabled.
Assignee | ||
Updated•8 months ago
|
Updated•8 months ago
|
Comment 13•8 months ago
|
||
The severity field is not set for this bug.
:jimm, could you have a look please?
For more information, please visit BugBot documentation.
Comment 14•8 months ago
|
||
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b403bb5b1c83 Remove transition prefixes from nightly too. r=jwatt
Comment 15•8 months ago
•
|
||
Backed out for causing mochitest failures in test_moz_prefixed_properties.html
- Backout link
- Push with failures
- Failure Log
- Failure line: TEST-UNEXPECTED-FAIL | dom/animation/test/mozilla/test_moz_prefixed_properties.html | Finished test - Finished test: Status: 1
And also this: https://treeherder.mozilla.org/logviewer?job_id=436772958&repo=autoland
Comment 16•8 months ago
|
||
There are a bunch of other failures, mochitest-plain, which are fixed by the above backout: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&revision=b403bb5b1c833d29961882402a090cf08f44c190&searchStr=mochitest-plain
Assignee | ||
Updated•7 months ago
|
Assignee | ||
Updated•7 months ago
|
Comment 17•7 months ago
|
||
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ffdf085e5e3c Remove transition prefixes from nightly too. r=jwatt
Comment 18•7 months ago
|
||
bugherder |
Updated•7 months ago
|
Updated•7 months ago
|
Updated•7 months ago
|
Description
•