stickers panel is not fully displayed on app.element.io
Categories
(Core :: CSS Parsing and Computation, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox-esr91 | --- | fixed |
firefox92 | --- | wontfix |
firefox93 | --- | wontfix |
firefox94 | --- | fixed |
People
(Reporter: ksenia, Assigned: emilio)
References
(Regression, )
Details
(Keywords: regression)
Attachments
(3 files, 1 obsolete file)
1.04 KB,
text/html
|
Details | |
791 bytes,
text/html
|
Details | |
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-esr91+
|
Details | Review |
This was originally reported in https://github.com/webcompat/web-bugs/issues/84751
To reproduce:
- Open the attached test case 84751.html in Firefox release/ Nightly and observe the block on the right
Expected:
The panel inside the iframe is fully visible
Actual:
The width of the panel is around 5px, so only a small part of it is visible
The issue only happens when I change the display property of a wrapper element dynamically with a setTimeout and works ok if the element is displayed by default. This is just to illustrate the behaviour - on the original site, it is changed on click.
From mozregression:
2:43.09 INFO: Narrowed integration regression window from [98a3f624, a74fee26] (3 builds) to [98a3f624, 58909c79] (2 builds) (~1 steps left)
2:43.09 INFO: No more integration revisions, bisection finished.
2:43.09 INFO: Last good revision: 98a3f624dd0822d471b46703d4cb0c8bafc1674b
2:43.09 INFO: First bad revision: 58909c79343bc9145173cfcd54c277cd343eb2a8
2:43.09 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=98a3f624dd0822d471b46703d4cb0c8bafc1674b&tochange=58909c79343bc9145173cfcd54c277cd343eb2a8
Reporter | ||
Comment 1•4 years ago
|
||
Reporter | ||
Comment 2•4 years ago
|
||
Updated•4 years ago
|
Updated•4 years ago
|
Reporter | ||
Comment 3•4 years ago
|
||
Hi Emilio, could you please take a look?
Reporter | ||
Updated•4 years ago
|
Reporter | ||
Updated•4 years ago
|
![]() |
||
Updated•4 years ago
|
Assignee | ||
Comment 4•4 years ago
|
||
Thanks for the excellent reduced test-case Ksenia! Of course, will take a look.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 5•4 years ago
|
||
Reporter | ||
Comment 6•4 years ago
|
||
Thanks!
Comment 9•4 years ago
|
||
bugherder |
Updated•4 years ago
|
Assignee | ||
Comment 12•4 years ago
|
||
Comment on attachment 9243928 [details]
Bug 1733475 - Ensure to copy over the viewport/font units invalidation bits when reusing reset properties. r=jfkthame,#style
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Simple invalidation correctness fix.
- User impact if declined: comment 0
- Fix Landed on Version: 94
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Copy some bits so that we invalidate correctly in some edge cases. Very simple fix.
- String or UUID changes made by this patch: none
Comment 14•4 years ago
|
||
Comment on attachment 9243928 [details]
Bug 1733475 - Ensure to copy over the viewport/font units invalidation bits when reusing reset properties. r=jfkthame,#style
Approved for 91.3esr.
Comment 15•4 years ago
|
||
bugherder uplift |
Description
•