If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Show less transparent homescreen / white window when hitting the home key / unlocking the phone

RESOLVED FIXED

Status

Firefox OS
Gaia::System
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: vingtetun, Assigned: vingtetun)

Tracking

unspecified
x86_64
Linux

Firefox Tracking Flags

(blocking-b2g:-, b2g18+ fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 702907 [details] [diff] [review]
Patch

This one is a very simple change that reduce the amount of transparent homescreen when you come back from an application to the homescreen. It also prevent many of the white application flash when going from the lockscreen to anywhere.

IMHO it really make the OS feels better.
Attachment #702907 - Flags: review?(etienne)
Josh I really think you want this one.

Asking blocking+ because without this patch very often when you came out of an application the homescreen is transparent before it is beeing rendered and that gives a very bad feeling about the product on this particular interaction.
blocking-b2g: --- → tef?
Flags: needinfo?(jcarpenter)
Comment on attachment 702907 [details] [diff] [review]
Patch

Review of attachment 702907 [details] [diff] [review]:
-----------------------------------------------------------------

::: apps/homescreen/js/homescreen.js
@@ +56,5 @@
> +        }
> +
> +        var element = document.createElement('div');
> +        element.id = 'repaint-helper';
> +        document.body.appendChild(element);

Just toggling the css display property wasn't enough to force the repaint?

::: apps/system/js/lockscreen.js
@@ +521,3 @@
>  
> +    var repaintTimeout = 0;
> +    var nextPaint= function() {

lint nit

@@ +552,5 @@
> +    this.dispatchEvent('will-unlock');
> +    currentFrame.addNextPaintListener(nextPaint);
> +    repaintTimeout = setTimeout(function ensureUnlock() {
> +      nextPaint();
> +    }, 400);

shouldn't this one be the same than kTransitionTimeout in the window manager?

Comment 3

5 years ago
Not a blocker since this is mostly cosmetic, but we will take in v1.0.x
blocking-b2g: tef? → -
tracking-b2g18: --- → +
/me can't wait for the updated version of the patch :)
Created attachment 703911 [details] [diff] [review]
Patch v2
Attachment #702907 - Attachment is obsolete: true
Attachment #702907 - Flags: review?(etienne)
Attachment #703911 - Flags: review?(etienne)
Comment on attachment 703911 [details] [diff] [review]
Patch v2

Review of attachment 703911 [details] [diff] [review]:
-----------------------------------------------------------------

r=me
(The nits are worth addressing ;))

::: apps/system/js/lockscreen.js
@@ +521,3 @@
>  
> +    var repaintTimeout = 0;
> +    var nextPaint= function() {

arrrrgggg, the lint error is still here ;)

::: apps/system/js/window_manager.js
@@ +71,4 @@
>    var screenElement = document.getElementById('screen');
>    var wrapperHeader = document.querySelector('#wrapper-activity-indicator');
>    var wrapperFooter = document.querySelector('#wrapper-footer');
> +  var kTransitionTimeout = 1000;

do we still need this bump since getting the repaint is sooo fast? :)
Attachment #703911 - Flags: review?(etienne) → review+
https://github.com/mozilla-b2g/gaia/commit/0c4274bf32ba13f5f82e2fc1720143d6e148f60d

1000 is to avoid as much as possible the transparent homescreen in bad cases.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Anything that reduces the amount of white-screen flashy-ness would be great. As you say, it makes the UI feel glitchy and cheap.

(In reply to Vivien Nicolas (:vingtetun) (:21) from comment #0)
> Created attachment 702907 [details] [diff] [review]
> Patch
> 
> This one is a very simple change that reduce the amount of transparent
> homescreen when you come back from an application to the homescreen.

Vivien, by transparent, do you mean the way in which the wallpaper shows first, but the other elements take a while to draw in?
Flags: needinfo?(jcarpenter)
Flags: needinfo?(21)
(In reply to Josh Carpenter [:jcarpenter] from comment #8)
> Anything that reduces the amount of white-screen flashy-ness would be great.
> As you say, it makes the UI feel glitchy and cheap.
> 
> (In reply to Vivien Nicolas (:vingtetun) (:21) from comment #0)
> > Created attachment 702907 [details] [diff] [review]
> > Patch
> > 
> > This one is a very simple change that reduce the amount of transparent
> > homescreen when you come back from an application to the homescreen.
> 
> Vivien, by transparent, do you mean the way in which the wallpaper shows
> first, but the other elements take a while to draw in?

Yep that's what I meant.
Flags: needinfo?(21)
status-b2g18: --- → fixed
:/ the 300 to 1000 ms is causing wrapper footer disappears before app starts to do transition.
See https://bugzilla.mozilla.org/show_bug.cgi?id=849441#c0 (7)
Blocks: 849441
You need to log in before you can comment on or make changes to this bug.