Closed
Bug 1199644
Opened 9 years ago
Closed 9 years ago
New widths for the two sections of about:privatebrowsing in Firefox 42
Categories
(Firefox :: Private Browsing, defect, P1)
Firefox
Private Browsing
Tracking
()
VERIFIED
FIXED
Iteration:
43.2 - Sep 7
People
(Reporter: Paolo, Assigned: Paolo)
References
Details
(Whiteboard: [fxprivacy])
Attachments
(4 files)
Updated•9 years ago
|
Flags: qe-verify?
Assignee | ||
Comment 1•9 years ago
|
||
Bug 1199644 - New widths for the two sections of about:privatebrowsing in Firefox 42. r=bgrins
Attachment #8654088 -
Flags: review?(bgrinstead)
Assignee | ||
Updated•9 years ago
|
status-firefox42:
--- → affected
Assignee | ||
Comment 2•9 years ago
|
||
Francesco, are the changes in the localization notes possible for Aurora?
We're reaching out to localizers about the width changes anyways, but if we can update the in-tree documentation it's better.
Flags: needinfo?(francesco.lodolo)
Comment 3•9 years ago
|
||
Yes, updating localization comment is fine (note that you'll still need to use the l10n flag to uplift, because it touches .dtd/.properties files).
Flags: needinfo?(francesco.lodolo)
Assignee | ||
Updated•9 years ago
|
Flags: qe-verify? → qe-verify+
Assignee | ||
Comment 4•9 years ago
|
||
(In reply to Francesco Lodolo [:flod] from comment #3)
> Yes, updating localization comment is fine (note that you'll still need to
> use the l10n flag to uplift, because it touches .dtd/.properties files).
I'm not familiar with the process, do you mean placing the late-l10n keyword on the bug?
Comment 5•9 years ago
|
||
Besides getting approval from release-driver, whoever commits this to aurora needs to add l10n={name-pf-release-driver-approving} to the commit message.
https://hg.mozilla.org/hgcustom/version-control-tools/file/f4661432e7c2/hghooks/mozhghooks/prevent_string_changes.py
Updated•9 years ago
|
Attachment #8654088 -
Flags: review?(bgrinstead) → review+
Comment 6•9 years ago
|
||
Comment on attachment 8654088 [details]
MozReview Request: Bug 1199644 - New widths for the two sections of about:privatebrowsing in Firefox 42. r=bgrins
https://reviewboard.mozilla.org/r/17609/#review15743
Fine with me as long as you have sign off for changing the entity value without updating the name
Assignee | ||
Comment 7•9 years ago
|
||
Comment on attachment 8654088 [details]
MozReview Request: Bug 1199644 - New widths for the two sections of about:privatebrowsing in Firefox 42. r=bgrins
Approval Request Comment
[Feature/regressing bug #]: New about:privatebrowsing landing page
[User impact if declined]: Sub-optimal design on a very visible page
[Describe test coverage new/current, TreeHerder]: Aurora-only version of bug 1198287, will do QA on the branch
[Risks and why]: Limited to the page
[String/UUID change made/needed]: Changed entity contents but not entity names
Attachment #8654088 -
Flags: approval-mozilla-aurora?
Comment 8•9 years ago
|
||
Comment on attachment 8654088 [details]
MozReview Request: Bug 1199644 - New widths for the two sections of about:privatebrowsing in Firefox 42. r=bgrins
New feature, we want to polish it, taking it.
Attachment #8654088 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 9•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
QA Contact: paul.silaghi
Comment 10•9 years ago
|
||
I've tried some random locales. Possible issues:
RU - private browsing section smaller than the tracking protection section
IT - tracking protection section higher
FR - private browsing section smaller than the tracking protection section, also both sections larger than usual
Flags: needinfo?(paolo.mozmail)
Comment 11•9 years ago
|
||
Comment 12•9 years ago
|
||
Assignee | ||
Comment 13•9 years ago
|
||
Francesco, I think we have now landed almost all the styling and text changes to about:privatebrowsing.
Can you recommend the best way to get in touch with localizers about the changes?
Flags: needinfo?(paolo.mozmail) → needinfo?(francesco.lodolo)
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(francesco.lodolo) → needinfo?(l10n)
Comment 14•9 years ago
|
||
I think I got this covered in https://groups.google.com/forum/#!topic/mozilla.dev.l10n/5wPH8xOLIVE, feel free to add to it.
Flags: needinfo?(l10n)
Assignee | ||
Comment 16•9 years ago
|
||
I just wanted to note that I've been in touch with Axel on the appearance of the about:privatebrowsing page in localized builds, but there doesn't seem to be any particular action left to do at this point. I think we are fine from a QA perspective.
Flags: needinfo?(paolo.mozmail)
You need to log in
before you can comment on or make changes to this bug.
Description
•