(photon) Update UI/Content for Empty screens and content for Android

VERIFIED FIXED in Firefox 56

Status

()

enhancement
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: wesley_huang, Assigned: jwu)

Tracking

(Blocks 2 bugs)

Trunk
Firefox 56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed, firefox57 verified)

Details

(Whiteboard: [FNC][SPT57.1][MVP])

User Story

Visual spec - https://drive.google.com/file/d/0B-9PIePlQZRldk9INm9TUXBkcWs/view?usp=sharing
Assets - https://drive.google.com/open?id=0B-9PIePlQZRlbGJ6RWp6bkl5RU0

Attachments

(4 attachments)

No description provided.
Assignee

Updated

2 years ago
Assignee: nobody → topwu.tw
Assignee

Updated

2 years ago
User Story: (updated)
Assignee

Comment 1

2 years ago
Hi Jack,

The information in `about:privatebrowsing` shows that private mode can provide both `Private browsing` and `Tracking protection`. You can check the attached file for screenshot.

But in visual spec, the empty screen in tabs tray only shows `Private browsing`, we have neither icon nor description about tracking protection. Should we also show it in our new visual design?

Furthermore, tracking protection can be enabled/disabled in Settings/Privacy/Tracking Protection. If the information shows in empty screen is what capabilities private mode can provide, maybe we should show/hide `Tracking protection` information based on user's preference.
Flags: needinfo?(jalin)
I'm going to NI Carol to get this on her radar.

Thank you
Flags: needinfo?(jalin) → needinfo?(chuang)
Assignee

Updated

2 years ago
Blocks: 1379656
Hi Jing- wei, 
I updated the visual spec.
on the visual spec: Modified Empty screen content - p.14

And to answer to your question, whether the tracking protection is enabled or disabled, please display the same information on the empty screen as current design. Thank you!
Flags: needinfo?(chuang) → needinfo?(topwu.tw)
Comment hidden (mozreview-request)
Assignee

Updated

2 years ago
Flags: needinfo?(topwu.tw)

Comment 6

2 years ago
mozreview-review
Comment on attachment 8887422 [details]
Bug 1366671 - Part 1: Duplicate empty screen resoueces for Photon.

https://reviewboard.mozilla.org/r/157288/#review164070
Attachment #8887422 - Flags: review?(walkingice0204) → review+

Comment 7

2 years ago
mozreview-review
Comment on attachment 8887423 [details]
Bug 1366671 - Part 2: Support empty screen in private tabs panel.

https://reviewboard.mozilla.org/r/157290/#review164072

::: mobile/android/base/java/org/mozilla/gecko/tabs/PrivateTabsPanel.java:64
(Diff revision 1)
> +                    geckoApp.autoHideTabs();
> +                }
> +            });
> +        }
> +
> +        emptyTabsFrame.getViewTreeObserver().addOnGlobalLayoutListener(new ViewTreeObserver.OnGlobalLayoutListener() {

it is likely redundant?
Attachment #8887423 - Flags: review?(walkingice0204) → review+
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Assignee

Updated

2 years ago
Keywords: checkin-needed

Comment 10

2 years ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5350f45b9724
Part 1: Duplicate empty screen resoueces for Photon. r=walkingice
https://hg.mozilla.org/integration/autoland/rev/2edde0aaa0f6
Part 2: Support empty screen in private tabs panel. r=walkingice
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/5350f45b9724
https://hg.mozilla.org/mozilla-central/rev/2edde0aaa0f6
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
+<!ENTITY private_tab_panel_title "Private-Browsing + Tracking Protection">

I've never seen Private Browsing spelled with an hyphen. Does this come from specs? The screenshots doesn't have an hyphen

+<!ENTITY private_tab_panel_description "Firefox will prevent you from being tracked and won\'t remember any history, but download files and new bookmarks will still be saved on your devices." >

You don't want Firefox hard-coded, there's an entity for that. At this point, you'll need a new string ID to fix it
https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices#Changing_existing_strings
Flags: needinfo?(topwu.tw)
(In reply to Francesco Lodolo [:flod] from comment #12)
> +<!ENTITY private_tab_panel_description "Firefox will prevent you from being
> tracked and won\'t remember any history, but download files and new
> bookmarks will still be saved on your devices." >

"download files" should be "downloaded files", not sure about "saved" (they were saved before, they'll be kept).

Is there a copy review process as part of the specs for Fennec?

Comment 14

2 years ago
Additionally, "on your devices" -> "to your device"? (or make changes in both so they match?)

https://transvision.mozfr.org/?recherche=files+and+new+bookmarks&repo=central&sourcelocale=en-US&locale=en-US&search_type=strings
Assignee

Comment 15

2 years ago
Sorry for late reply. And after checking with the designer :carol, the strings  are copied from "about:privatebrowsing" web page, and that's why we didn't have string review before.

But some typos occurred while duplicating the strings(the hyphen and "ed" things), also the content in "about:privatebrowsing" is changed after the duplication. We would file another bug for these strings fixing.
Flags: needinfo?(topwu.tw)
Assignee

Updated

2 years ago
Depends on: 1382980

Comment 16

2 years ago
Backout by archaeopteryx@coole-files.de:
https://hg.mozilla.org/mozilla-central/rev/1acc7a08d914
Backed out changeset 2edde0aaa0f6 
https://hg.mozilla.org/mozilla-central/rev/874cdf9f9c9e
Backed out changeset 5350f45b9724 on request from flod. r=backout a=backout
Backed this out as requested by :flod.
Flags: needinfo?(topwu.tw)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Assignee

Updated

2 years ago
Flags: needinfo?(topwu.tw)
Keywords: checkin-needed

Comment 20

2 years ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/aeb5d6a6ddd3
Part 1: Duplicate empty screen resoueces for Photon. r=walkingice
https://hg.mozilla.org/integration/autoland/rev/9cb73bdb10a7
Part 2: Support empty screen in private tabs panel. r=walkingice
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/aeb5d6a6ddd3
https://hg.mozilla.org/mozilla-central/rev/9cb73bdb10a7
Status: REOPENED → RESOLVED
Last Resolved: 2 years ago2 years ago
Resolution: --- → FIXED

Updated

2 years ago
Whiteboard: [FNC][SPT_57.1][MVP]

Updated

2 years ago
Whiteboard: [FNC][SPT_57.1][MVP] → [FNC][SPT57.1][MVP]

Comment 22

2 years ago
Posted image Screenshot
(In reply to Carol Huang [:Carol] from comment #3)
> Hi Jing- wei, 
> I updated the visual spec.
> on the visual spec: Modified Empty screen content - p.14

The image differs from the specs but in the meaning it is the same. Can you please confirm it is the right one? 

Pixel - Android 8.0 - Nightly 2017-08-13
Flags: needinfo?(chuang)
hi Ioana!
Recently we decide not to use any image for this screen. I will update the spec and reopen this bug to fix it!
Thanks!!
Flags: needinfo?(chuang)
Assignee

Updated

2 years ago
Blocks: 1390420

Comment 24

2 years ago
Verified the new private mode empty screen design matches the new specs.
Build: Nightly 57 (2017-08-23)
Devices:
Xiaomi Mi Pad 2 (Android 5.1)
Motorola Nexus 6 (Android 7.0)
HTC Desire 820 (Android 6.0.1)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.