Black rect to left of the about:start watermark displays when scrolling

VERIFIED FIXED in Firefox 30

Status

()

defect
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: jimm, Assigned: roc)

Tracking

26 Branch
mozilla30
x86_64
Windows 8.1
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox27 unaffected, firefox28 unaffected, firefox29 affected, firefox30+ verified)

Details

(Whiteboard: [metro] [qa+])

Attachments

(1 attachment)

Reporter

Description

5 years ago
STR:

1) Load about:start in metrofx 
2) touch scroll to the right

result: a black rect is displayed briefly to the left of the watermark image block.
Reporter

Updated

5 years ago
Reporter

Comment 1

5 years ago
Note, it helps if you clear history to remove all top site tiles. The rect shows up in the watermark layer, so top site tiles can obscure it.
Reporter

Comment 2

5 years ago
Also, it only happens on the first about:start pan from the left bounds. Interim scrolls don't reproduce.
Reporter

Comment 3

5 years ago
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #58)
> (In reply to Jim Mathies [:jimm] from comment #57)
> > This is a regression from bug 967587, filed bug 969354.
> 
> That makes sense. You can probably work around it in about:start by making
> the image slightly transparent or something like that.
Reporter

Updated

5 years ago
Whiteboard: [metro] → [metro][release28]
Reporter

Updated

5 years ago
Depends on: 970990
Nothing obvious springs to mind as to what's causing the problem here.

If you delete the descendants of <body> before scrolling, does the bug still occur? If not, then which element triggers it? If so, at least we have a reduced testcase.
Reporter

Comment 6

5 years ago
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #5)
> Nothing obvious springs to mind as to what's causing the problem here.
> 
> If you delete the descendants of <body> before scrolling, does the bug still
> occur? If not, then which element triggers it? If so, at least we have a
> reduced testcase.

If you remove everything, leaving only the body with the watermark with a long width set on it, the problem goes away. Add back the first hbox -

http://mxr.mozilla.org/mozilla-central/source/browser/metro/base/content/startui/Start.xul#44

with nothing in it, the problem comes back.

Note, rsilveira may have found a work around for this in bug 971124.
Reporter

Updated

5 years ago
No longer blocks: metrov1backlog
Whiteboard: [metro][release28] → [metro]
Posted patch proper fixSplinter Review
Needs manual testing. Jim, can you test if this fixes Metro properly? I'll test it on Android/B2G.
Assignee: nobody → roc
Attachment #8377355 - Flags: review?(matt.woodrow)
Flags: needinfo?(jmathies)
Reporter

Comment 8

5 years ago
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #7)
> Created attachment 8377355 [details] [diff] [review]
> proper fix
> 
> Needs manual testing. Jim, can you test if this fixes Metro properly? I'll
> test it on Android/B2G.

Reapplied our old watermark css and I can confirm this fixes the black rect problem.
Flags: needinfo?(jmathies)
Attachment #8377355 - Flags: review?(matt.woodrow) → review+
https://hg.mozilla.org/mozilla-central/rev/68b905ed5fc4
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
Is this safe to uplift to branches?
Flags: needinfo?(roc)
Sorry, this should be unaffected on 28 if it's a blocker for 967587 which is also marked as such.
For testing and verification.  Reopen if defects are found.
Flags: needinfo?(kamiljoz)
I discussed this with :rsilveira via IRC and we're not using the same watermark anymore. Based on Jims comment #8, I'm marking this as verified.
Status: RESOLVED → VERIFIED
Flags: needinfo?(kamiljoz)
Whiteboard: [metro] → [metro] [qa+]

Updated

5 years ago
Depends on: 978490
Flags: needinfo?(roc)
You need to log in before you can comment on or make changes to this bug.