Closed Bug 1174242 Opened 5 years ago Closed 5 years ago

Final copy for Tracking Protection

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: antlam, Unassigned)

References

Details

Attachments

(2 files)

Matej,

Could we get your eyes on the copy for Tracking Protection? I've been taking a crack at the copy (horrible cracks) but it'd be great to get some feedback. Especially since we have to be more concise on Mobile (small screens and attn spans).


On Android we need copy for:

 - Doorhanger
 - "Onboarding" slide
 - Settings

I basically appropriated what I saw from Desktop side, but let me know what you think!
Attached image prev_droid_TP.png
Attaching screenshots for visual reference.

(ignore the Site ID doorhanger above on the left)
Flags: needinfo?(matej)
If this changes Settings, be sure to update testSettingsMenuItems so we don't have another bug 1174072.
Hi Anthony,

I don't know if there was anything specific you had concerns about, but I think the copy looks good as is. The only thing I noticed is that "Do Not Track" should be capitalized.
Flags: needinfo?(matej)
(In reply to Matej Novak [:matej] from comment #3)
> Hi Anthony,
> 
> I don't know if there was anything specific you had concerns about, but I
> think the copy looks good as is. 

We're using this bug to keep track of the copy for the entire feature as it goes across platforms. I'm not sure where I got that (the copy in the mocks) from but I wanted to NI you so we can have the same consistency through desktop and mobile.

I'd like your help with the following :)

On Android we need copy for:

 - Doorhanger
 - "Onboarding" slide
 - Settings

Perhaps Mike can post the desktop designs too so you can look at all the copy together?
Flags: needinfo?(mmaslaney)
Flags: needinfo?(matej)
Hi Anthony,

I'm not clear whether you just need me to review something, or if you also need me to create copy for those elements.

If the latter, please fill out a project initiation for so we can track it: https://bugzilla.mozilla.org/form.creative

Either way, I'm going to clear my flag for now, until there's something for me to do. Thanks.
Flags: needinfo?(matej)
Matej, from the google doc that Aislinn put together, you mentioned you wanted to take the copy work into this bug. 

Let's sort out the copy for all the slides you saw in the doc here.

From Android, we have those 3 screens and in them are my cracks at copy. Could we get your eyes on those and comments in here?

Thanks.
Flags: needinfo?(matej)
(In reply to Anthony Lam (:antlam) from comment #6)
> Matej, from the google doc that Aislinn put together, you mentioned you
> wanted to take the copy work into this bug. 
> 
> Let's sort out the copy for all the slides you saw in the doc here.
> 
> From Android, we have those 3 screens and in them are my cracks at copy.
> Could we get your eyes on those and comments in here?
> 
> Thanks.

I think Matej just wanted us to needinfo in here when it can be looked at. I pinged Javaun to see if we want a seperate desktop bug since this request ideally should encompass both but this bug is only tracking on the android queue. If we can copy all final copy in the google doc but use bugzilla for comments/questions that would be easiest for the desktop side as after Matej is done, we may need branding and some other folks to weigh in.
Matej - all screens are now in the google doc here: 
https://docs.google.com/a/mozilla.com/document/d/134Ez8FpoMVkJ72a4ux8t6mPVNTU8iNPwAa5On4cUQwU/edit?usp=sharing

Please add you copy to the copy column. You can write over what is currently there.
I've left some comments and edits in the doc. Let me know if you have any questions or need anything else. Thanks.
Flags: needinfo?(matej)
(In reply to Matej Novak [:matej] from comment #9)
> I've left some comments and edits in the doc. Let me know if you have any
> questions or need anything else. Thanks.

followed up with the comments in the doc
Liuche, it looks like the changes to our copy is rather minimal, would you mind taking a quick pass and verify the copy in the google doc is the same as what we're using? (capitalization, punctuation, etc)

After that I think we can close this bug
Flags: needinfo?(liuche)
(In reply to agrigas from comment #10)
> (In reply to Matej Novak [:matej] from comment #9)
> > I've left some comments and edits in the doc. Let me know if you have any
> > questions or need anything else. Thanks.
> 
> followed up with the comments in the doc

Thanks. I made a change to first paragraph in the gdoc.
A few comments:
I assume we're going by the descriptor strings (not the images). The preference mock uses "Private browsing" instead of "Private Browsing" but that's minor.

I see a few inconsistencies between our styling.
- Currently in Tab Queues, we don't use periods, but the mocks for the Private Browsing overlay does. We also don't have any capitalization in the subtext of TQ (which actually looks really, really weird IMO). See screenshot. We should make that consistent, and maybe that involves changing the styling in TQ.

- Tab Queues is also partial width, whereas the Tracking protection notification extends to the full width of the screen. Might be intentional though.
Flags: needinfo?(liuche) → needinfo?(alam)
(In reply to Chenxia Liu [:liuche] from comment #13)
> Created attachment 8627392 [details]
> Screenshot: Inconsistency with Tab Queue
> 
> A few comments:
> I assume we're going by the descriptor strings (not the images). The
> preference mock uses "Private browsing" instead of "Private Browsing" but
> that's minor.
> 
> I see a few inconsistencies between our styling.
> - Currently in Tab Queues, we don't use periods, but the mocks for the
> Private Browsing overlay does. We also don't have any capitalization in the
> subtext of TQ (which actually looks really, really weird IMO). See
> screenshot. We should make that consistent, and maybe that involves changing
> the styling in TQ.
> 
> - Tab Queues is also partial width, whereas the Tracking protection
> notification extends to the full width of the screen. Might be intentional
> though.

Yeah, follow the column that has the text, that's what Matej editted. The images I attached originally to give him context.

Tracking protection notification should behave the same as Tab Queue and only extend partial as well.
Flags: needinfo?(alam) → needinfo?(liuche)
Ok, that's clear then. The strings for Android look good, so closing this bug.

The final copy is in comment 8.
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(mmaslaney)
Flags: needinfo?(liuche)
Resolution: --- → FIXED
See Also: → 1179850
Blocks: 1175970
QA Contact: teodora.vermesan
You need to log in before you can comment on or make changes to this bug.