Closed
Bug 1174242
Opened 9 years ago
Closed 9 years ago
Final copy for Tracking Protection
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
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!
Reporter | ||
Comment 1•9 years ago
|
||
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.
Comment 3•9 years ago
|
||
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)
Reporter | ||
Comment 4•9 years ago
|
||
(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)
Comment 5•9 years ago
|
||
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)
Reporter | ||
Comment 6•9 years ago
|
||
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)
Comment 7•9 years ago
|
||
(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.
Comment 8•9 years ago
|
||
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.
Comment 9•9 years ago
|
||
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)
Comment 10•9 years ago
|
||
(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
Reporter | ||
Comment 11•9 years ago
|
||
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)
Comment 12•9 years ago
|
||
(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.
Comment 13•9 years ago
|
||
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)
Reporter | ||
Comment 14•9 years ago
|
||
(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)
Comment 15•9 years ago
|
||
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: 9 years ago
Flags: needinfo?(mmaslaney)
Flags: needinfo?(liuche)
Resolution: --- → FIXED
Updated•9 years ago
|
QA Contact: teodora.vermesan
Assignee | ||
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•