[graphene] show a "install" screen while we install the app

RESOLVED FIXED

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: paul, Assigned: paul)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: graphene-larch)

Attachments

(1 attachment, 1 obsolete attachment)

10.67 KB, patch
fabrice
: review+
Details | Diff | Splinter Review
(Assignee)

Description

3 years ago
The original idea was to only show the native window once the app is fully loaded. It's supposed to be fast enough when the app is in cache, but when it's not, the window is hidden for too long.

Until we manage to ship Graphene with a preloaded version of the app, we should show a "install" screen when the app is being downloaded for the first time.
(Assignee)

Comment 1

3 years ago
Created attachment 8640349 [details] [diff] [review]
v1
Attachment #8640349 - Flags: review?(fabrice)
(Assignee)

Updated

3 years ago
Blocks: 1115098
Comment on attachment 8640349 [details] [diff] [review]
v1

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

::: b2g/chrome/content/shell.html
@@ +73,5 @@
>  #endif
> +#else
> +    <div id="installing">
> +      <div id="throbber"></div>
> +      <div>Installing...</div>

:( l10n...
Attachment #8640349 - Flags: review?(fabrice) → review+
(Assignee)

Comment 3

3 years ago
Created attachment 8640966 [details] [diff] [review]
v1.1

Using a properties file instead of a dtd because shell.html is not a xml file.
Assignee: nobody → paul
Attachment #8640349 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8640966 - Flags: review?(fabrice)
Attachment #8640966 - Flags: review?(fabrice) → review+
Landed in mozilla-central via bug 1204965.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.