Closed Bug 1020530 Opened 10 years ago Closed 10 years ago

[email] Message list: temporary fuzzy areas when scrolling

Categories

(Core :: Graphics, defect)

32 Branch
x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED INVALID
Tracking Status
b2g-v2.0 --- affected
b2g-v2.1 --- affected

People

(Reporter: jrburke, Unassigned)

References

Details

(Keywords: regression)

Attachments

(2 files)

Attached image fuzzy-scroll.png (deleted) —
To reproduce:

* get latest master
* make sure to have a few screens worth of messages in the inbox.
* scroll in the message list for the inbox.

Some regions will show fuzzy rendering for a moment while scrolling, which are eventually replaced with the hi-res, proper display of the message. So it is just temporary.

Attached is a screenshot that was captured mid-scroll.

The cause appears to be the result of bug 993473. Apparently there are still some issues to be sorted out with that low-res tiling. Logging a bug to track the impact on email, as it is likely QA or others will file an issue.

As it behaves currently on master, it looks more like a rendering glitch since eventually it snaps to a hi-res display. At least for the email case, I believe the white was preferable as it blended in more with the background, and was harder to see. With the fuzzy black regions it seems much more obvious.

The fix could be that email needs to do adjustments to its code, but could also be a combo fix with some platform changes. Hopefully this bug can be used to track any bug dependencies.
[Blocking Requested - why for this release]:
I hit this issue as well.  It's either completely white or fuzzy.  You can't see the load more option nor email after a certain point.

It doesn't occur in 2.0; this is a regression.
blocking-b2g: --- → 2.1?
QA Contact: ckreinbring
Regression window:

Last working
Build ID: 20140529133000
Gaia: b669dd2cc321f37cebc7081a79b968cac36b4200
Gecko: b85b57f05fda
Platform Version: 32.0a1
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

First broken
Build ID: 20140529193003
Gaia: 26d8fcab9b61f46451600f39c51e0387ef3c4f88
Gecko: 38c5e21a80fa
Platform Version: 32.0a1
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Working Gaia / Broken Gecko = Repro
Broken Gaia / Working Gecko = No repro
Gecko push log: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=b85b57f05fda&tochange=38c5e21a80fa


B2G-Inbound window

Last working
Build ID: 20140529173003
Gaia: 1477f488ddc8746b8efe8ba32a3a35a545a01422
Gecko: 0ad306bee50e
Version: 32.0a1 (Master)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

First broken
Build ID: 20140529203002
Gaia: af14823a08bd20efbd83157e0f952929a5ca7773
Gecko: 6b4c05be821f
Version: 32.0a1 (Master)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Working Gaia / Broken Gecko = Repro
Broken Gaia / Working Gecko = No repro
Gecko pushlog: http://hg.mozilla.org/integration/b2g-inbound/pushloghtml?fromchange=0ad306bee50e&tochange=6b4c05be821f
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(jmitchell)
Broken by bug 950208 - Email visual refresh ?
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmitchell) → needinfo?(jrburke)
I do not believe this is a result of the email visual refresh. The scrolled message list in particular changed very little. The artifacts look like what is expected as a result of the low res tiling decision in bug 993473.

This is more about if that low res tiling option makes sense for all apps.

:nhirata, for the video in Comment 2, that all-white appearance looks like a temporary regression in the graphics code, but I cannot seem to reproduce now with the latest pvt build on flame:

Gaia      c72257b2d27135bfcd68e89dd584182797784016
Gecko     https://hg.mozilla.org/mozilla-central/rev/06ac51c2b8a8
BuildID   20140724040205
Version   34.0a1
ro.build.version.incremental=110
ro.build.date=Fri Jun 27 15:57:58 CST 2014

but if it does continue to show, the all-white I suspect is a different issue than the fuzzy black text as a result of the low-res tiling decision.

If this bug then is just about that tiling decision, then I will likely just close as invalid as it is just a result of that design choice, and it seems to occur less frequently with the latest pvt build.
Flags: needinfo?(jrburke)
Naoki - Shouldn't this be nomed for 2.0, not 2.1, since this reproduces on 2.0?
Component: Gaia::E-Mail → Graphics
Flags: needinfo?(nhirata.bugzilla)
Product: Firefox OS → Core
Version: unspecified → 32 Branch
Blocks: 993473
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][lead-review+]
Low res tiling is either on or off.  We're choosing between low res tiling or checkerboarding (e.g., blank, or potentially seeing the stuff underneath the content that couldn't be painted in time).
So, this bug can either be:
1. Ask product/UX to turn off low res tiling everywhere
1a. Ask product/UX to have us expose a pref, and allow each device to choose if low res is on or off everywhere
2. Introduce a method for each app to choose if it wants low res tiling on or off.  I imagine this would be a property, but I'm not sure where it would live.  Perhaps some kind of a background-if-unavailable that would also let us specify what kind of checkerboarding texture (rather than blank) we may want, and also let us choose low res as an option.
I confused the white out issues with the blurred issue.  The white out was permanent for me at the time I was testing.  Same with the blurred effect in yesterday's build which only occurred on 2.1 for me.

I agree; I cannot reproduce the whiteout issue in 2.1 any more in today's build; to note I am using 319 MB on the flame.

Because James's initial point is that it's temporary, I am withdrawing my nomination to block.
blocking-b2g: 2.1? → ---
Flags: needinfo?(nhirata.bugzilla)
Note that we did have bug 1041608 for a few days, where you would get stuck in low res mode, since bug 1022612 in the 7/21 build.
(In reply to James Burke [:jrburke] from comment #5)
> I do not believe this is a result of the email visual refresh. The scrolled
> message list in particular changed very little. The artifacts look like what
> is expected as a result of the low res tiling decision in bug 993473.

Agreed.

> This is more about if that low res tiling option makes sense for all apps.

This is something that should be asked of the UX team. Thus far Gordon has said that low-res is preferable to blank white in all the cases he's seen, but maybe he hasn't looked at this case specifically.

(In reply to Milan Sreckovic [:milan] from comment #7)
> 2. Introduce a method for each app to choose if it wants low res tiling on
> or off.  I imagine this would be a property, but I'm not sure where it would
> live.  Perhaps some kind of a background-if-unavailable that would also let
> us specify what kind of checkerboarding texture (rather than blank) we may
> want, and also let us choose low res as an option.

This option also involves asking UX if this is acceptable.

So it seems there are only two options here:
(1) ask UX if low-res is ok in the email message list.
(2) resolve invalid.

Going with (1) for now but James, feel free to clear the needinfo and go with (2) if you feel the latest builds are fine (per what you said in comment 5).
Flags: needinfo?(gbrander)
Marking resolved, invalid, as this behavior was explicitly asked for by UX. While I think it makes it look like a drawing error, at least in the email case with lists of black text on white backgrounds, it does seem like I see it a bit less, on the flame, than before.
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(gbrander)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: