KidFox: First-run screen for restricted profile

VERIFIED FIXED in Firefox 42

Status

()

defect
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: sebastian, Assigned: sebastian)

Tracking

42 Branch
Firefox 42
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox42 verified)

Details

Attachments

(2 attachments, 6 obsolete attachments)

We'd like to show a different first-run screen if the user is using a restricted profile.

Bug 1180795 disabled the first-run screen for restricted profiles.
Status: NEW → ASSIGNED
Posted image restricted-first-run.png (obsolete) —
This is a first WIP version. I return a different list of FirstrunPanels for a restricted profile. This way we can use a completely different layout for the first run screen but keep all the logic. Only requirement is that we keep the blue tab bar at the top (Title(s) can be different). We can even have multiple pages (later).
Posted image kidfox_firstrun_wip.png (obsolete) —
This is an updated WIP version based on the UX work. Some of the metrics might need tweaking.

@Robin: Is this going into the right direction?
Attachment #8644456 - Flags: feedback?(randersen)
Posted image n7_firstrun.png (obsolete) —
For comparison: This is the same screen on a smaller Nexus 7.
Depends on: bug 1189920 - I'll need the SUMO page for the "Learn more" link.
Depends on: 1189920
Posted image kidfox_firstrun_white.png (obsolete) —
White version.
Attachment #8644456 - Attachment is obsolete: true
Attachment #8644456 - Flags: feedback?(randersen)
Posted patch 1191208_firstrun_wip.patch (obsolete) — Splinter Review
This is the updated WIP patch (<-> kidfox_firstrun_white.png).

Already asking for feedback to reduce review time tomorrow. :)
Attachment #8643677 - Attachment is obsolete: true
Attachment #8644507 - Flags: review?(margaret.leibovic)
Attachment #8644507 - Flags: review?(ally)
Comment on attachment 8644507 [details] [diff] [review]
1191208_firstrun_wip.patch

Chenxia knows this code well, so also asking her to take a look.
Attachment #8644507 - Flags: feedback?(liuche)
All the kidfox specific bits look good to me. Deferring to Chenxia as the owner of first run. :)
Screenshot of updated first run.
Attachment #8643673 - Attachment is obsolete: true
Attachment #8644469 - Attachment is obsolete: true
Attachment #8644479 - Attachment is obsolete: true
Final updated patch for review. :)
Attachment #8644507 - Attachment is obsolete: true
Attachment #8644507 - Flags: review?(margaret.leibovic)
Attachment #8644507 - Flags: review?(ally)
Attachment #8644507 - Flags: feedback?(liuche)
Attachment #8644885 - Flags: review?(margaret.leibovic)
Attachment #8644885 - Flags: review?(liuche)
Comment on attachment 8644885 [details] [diff] [review]
1191208-kidfox-firstrun.patch

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

In the spirit of rushing to land strings, I think this looks good enough to land, and we can address any feedback Chenxia has in a follow-up bug.

::: mobile/android/base/firstrun/RestrictedWelcomePanel.java
@@ +42,5 @@
> +
> +        root.findViewById(R.id.welcome_browse).setOnClickListener(new View.OnClickListener() {
> +            @Override
> +            public void onClick(View v) {
> +                Telemetry.sendUIEvent(TelemetryContract.Event.ACTION, TelemetryContract.Method.BUTTON, "firstrun-browser");

I don't think we should keep this probe the same as the probe for the browse link in the normal first run experience, since the context is different. Let's just remove this probe for now, and we can add in a probe if there's some explicit question we're trying to answer.

::: mobile/android/base/resources/layout/restricted_firstrun_welcome_fragment.xml
@@ +28,5 @@
> +            <TextView
> +                android:layout_width="350dp"
> +                android:layout_height="wrap_content"
> +                android:layout_gravity="center"
> +                android:background="#1aaa86"

Maybe we should add this to the color palette, in case we want to use it in other places in this restricted version of Firefox.
Attachment #8644885 - Flags: review?(margaret.leibovic) → review+
url:        https://hg.mozilla.org/integration/fx-team/rev/0d99513542f3233cccb0e04c7b337241b4dba68c
changeset:  0d99513542f3233cccb0e04c7b337241b4dba68c
user:       Sebastian Kaspari <s.kaspari@gmail.com>
date:       Fri Aug 07 21:01:00 2015 +0200
description:
Bug 1191208 - Custom first-run screen for restricted profiles. r=margaret
Blocks: 1192446
Comment on attachment 8644885 [details] [diff] [review]
1191208-kidfox-firstrun.patch

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

::: mobile/android/base/locales/en-US/android_strings.dtd
@@ +9,5 @@
>  <!ENTITY  onboard_start_message3 "Browse with &brandShortName;">
>  <!ENTITY  onboard_start_subtext3 "Make your mobile Web browsing experience truly your own.">
>  <!ENTITY  onboard_start_button_account "Sign in to &brandShortName;">
>  <!ENTITY  onboard_start_button_browser "Start Browsing">
> +<!ENTITY  onboard_start_restricted "Stay safe and in control with this simplified version of Firefox.">

We want this to match the channel right? In that case, we should use &brandName. I'll file and fix this, in bug 1192446.
Attachment #8644885 - Flags: review?(liuche) → review+
https://hg.mozilla.org/mozilla-central/rev/0d99513542f3
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Verified that the first-run screen is displayed on Nightly and Aurora
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.