Closed
Bug 1471181
Opened 7 years ago
Closed 7 years ago
[Twitter] Rendering issue inside the Reply popup
Categories
(Core :: Web Painting, defect, P2)
Tracking
()
VERIFIED
DUPLICATE
of bug 1472465
mozilla63
| Tracking | Status | |
|---|---|---|
| firefox-esr52 | --- | unaffected |
| firefox-esr60 | --- | unaffected |
| firefox61 | + | wontfix |
| firefox62 | + | verified |
| firefox63 | + | verified |
People
(Reporter: asoncutean, Assigned: mikokm)
References
Details
(Keywords: regression)
Attachments
(1 file)
|
101.92 KB,
image/png
|
Details |
[Affected versions]:
- 61.0 (20180621125625)
- 62.0b3 (20180625141512)
- 63.0a1 (2018-06-25)
[Affected platforms]:
- Ubuntu 16.04 x64
- Windows 10 x64
- Windows 7 x32
- Mac OS 10.11
[Steps to reproduce]:
1. Go to https://twitter.com/
2. Click on the Reply button
3. Type something inside the Reply to [..] popup
4. Observe the Reply button inside the popup
[Additional steps]:
- This issue seems to occur often if first, the Retweet popup is opened and closed, before step 2 is executed.
[Expected result]:
- The Reply button is clear, active and clickable.
[Actual result]:
- Most of the time the Reply button remains grey out, or partially grey out (see screenshots: https://drive.google.com/open?id=1FgQybQDWyYC_p-Y3tJzV-f5e7z4z7S8U and https://drive.google.com/open?id=19n9xxNeR1LJxFxBeItO9BqMMLjEmFuDD).
[Regression range]:
- I will try to determine if this is a regression asap.
[Additional Notes]:
- This rendering issue doesn’t affect in any way the functionality of the Reply button, if hover over it, the button is not grey out anymore.
- This behavior is intermittent.
Comment 1•7 years ago
|
||
Sounds like this is an issue where we may want to contact Twitter, might be a change on their end.
| Reporter | ||
Comment 2•7 years ago
|
||
Using mozregression, I ended up to the following:
Last good revision: 0f807beef229718c015553bac09e4f638ca2add1
First bad revision: 665843c6cfd1a3049128597afbef8a68bcc1e086
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=0f807beef229718c015553bac09e4f638ca2add1&tochange=665843c6cfd1a3049128597afbef8a68bcc1e086
Keywords: regressionwindow-wanted → regression
Updated•7 years ago
|
Component: Layout: View Rendering → Layout: Web Painting
Flags: needinfo?(mikokm)
Updated•7 years ago
|
Blocks: 1442190
status-firefox-esr52:
--- → unaffected
status-firefox-esr60:
--- → unaffected
tracking-firefox61:
--- → +
tracking-firefox62:
--- → +
tracking-firefox63:
--- → +
Updated•7 years ago
|
Version: Trunk → 61 Branch
Comment 3•7 years ago
|
||
Anca, can you put the screenshots in this bug as attachments? Thanks!
Flags: needinfo?(anca.soncutean)
Updated•7 years ago
|
Assignee: nobody → mikokm
Priority: -- → P2
| Reporter | ||
Comment 4•7 years ago
|
||
Flags: needinfo?(anca.soncutean)
| Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(mikokm)
| Assignee | ||
Comment 6•7 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #5)
> Hey Miko, have you had a chance to look into this?
Not yet unfortunately. There was a work week last week, where this part of the code got a lot of changes. The upcoming transform flattening (bug 1462672), will cause almost a complete rewrite of the opacity flattening code, as they share a lot of functionality.
I will look into this on Monday.
Flags: needinfo?(mikokm)
| Assignee | ||
Comment 7•7 years ago
|
||
Fixed in bug 1472465.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Updated•7 years ago
|
Target Milestone: --- → mozilla63
Comment 8•7 years ago
|
||
This is tracked in bug 1472465, where it looks like we have a fix which should land soon (later this week) in beta.
Comment 9•7 years ago
|
||
Anca, can you verify that this is fixed on Nightly and Beta now?
Flags: needinfo?(anca.soncutean)
| Reporter | ||
Comment 10•7 years ago
|
||
Yes, the issue is no longer reproducible on Firefox 63.0a1 (2018-07-22) and Firefox 62.0b10 (20180719140244) under Windows 10 x64, macOS 10.13 and Ubuntu 16.04 x32.
Flags: needinfo?(anca.soncutean)
Updated•7 years ago
|
Status: RESOLVED → VERIFIED
Comment 11•7 years ago
|
||
Glad to see this fix riding the 62 train. I think we can live with it for another month on 61, though.
You need to log in
before you can comment on or make changes to this bug.
Description
•