If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Unnecessary whitespace displayed after aboutPrivateBrowsing.info.saved.emphasize in about:privatebrowing

RESOLVED FIXED in Firefox 48

Status

()

Firefox
Private Browsing
P1
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: flod, Assigned: rickychien)

Tracking

(Blocks: 1 bug)

unspecified
Firefox 49
Points:
---
Dependency tree / graph
Bug Flags:
qe-verify ?

Firefox Tracking Flags

(firefox48 fixed, firefox49 fixed)

Details

(Whiteboard: [fxprivacy])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

An unexpected whitespace is currently displayed after "will save"
http://hg.mozilla.org/mozilla-central/rev/463fc6d36ff5#l3.77

Proof is that the next string "your:" doesn't start with a space.
http://hg.mozilla.org/mozilla-central/rev/463fc6d36ff5#l6.26

Layout should not add spaces in the middle of a sentence, that's something that strings should do.
Blocks: 1216897
Whiteboard: [fxprivacy] [triage]
Ricky, could you submit a quick patch for this? Thanks!
Assignee: nobody → rchien
Flags: needinfo?(rchien)
Priority: -- → P3
Whiteboard: [fxprivacy] [triage] → [fxprivacy]
(Assignee)

Comment 2

a year ago
Sure.
Flags: needinfo?(rchien)
(Assignee)

Comment 3

a year ago
Created attachment 8745866 [details]
MozReview Request: Bug 1267434 - Unnecessary whitespace displayed in about:privatebrowing; r?Gijs

Review commit: https://reviewboard.mozilla.org/r/49135/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/49135/
Attachment #8745866 - Flags: review?(gijskruitbosch+bugs)
https://reviewboard.mozilla.org/r/49135/#review45951

::: browser/locales/en-US/chrome/browser/aboutPrivateBrowsing.dtd:20
(Diff revision 1)
>  <!ENTITY aboutPrivateBrowsing.info.searches              "searches">
>  <!ENTITY aboutPrivateBrowsing.info.cookies               "cookies">
>  <!ENTITY aboutPrivateBrowsing.info.temporaryFiles        "temporary files">
>  <!ENTITY aboutPrivateBrowsing.info.saved.before          "Firefox ">
>  <!ENTITY aboutPrivateBrowsing.info.saved.emphasize       "will save">
> -<!ENTITY aboutPrivateBrowsing.info.saved.after           "your:">
> +<!ENTITY aboutPrivateBrowsing.info.saved.after           " your:">

And sadly at this point you'll need a new key.
(Assignee)

Comment 5

a year ago
Comment on attachment 8745866 [details]
MozReview Request: Bug 1267434 - Unnecessary whitespace displayed in about:privatebrowing; r?Gijs

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/49135/diff/1-2/
(Assignee)

Comment 6

a year ago
Oops! thank you for quickly reply, aboutPrivateBrowsing.dtd was updated.

Updated

a year ago
Attachment #8745866 - Flags: review?(gijskruitbosch+bugs) → review+

Comment 7

a year ago
Comment on attachment 8745866 [details]
MozReview Request: Bug 1267434 - Unnecessary whitespace displayed in about:privatebrowing; r?Gijs

https://reviewboard.mozilla.org/r/49135/#review46051

r=me

Flod, what do you want to do about uplifts / 48 ? The original patch made 48, and this obviously won't anymore unless we uplift.

Comment 8

a year ago
(In reply to :Gijs Kruitbosch from comment #7)
> Flod, what do you want to do about uplifts / 48 ? The original patch made
> 48, and this obviously won't anymore unless we uplift.
Flags: needinfo?(francesco.lodolo)
I'm a bit torn, but I think we should try to land this as soon as possible in aurora.

For Italian I had to do a trick like moving the ":" to the string before, French now has two spaces.

We might want to add a note to this bug in the report we send out to localizers (@jeff: there should be a string freeze break to fix the localizability of the new about:privatebrowsing window)
Flags: needinfo?(francesco.lodolo)

Comment 10

a year ago
https://hg.mozilla.org/integration/fx-team/rev/099e2bf3ce03

Comment 11

a year ago
Comment on attachment 8745866 [details]
MozReview Request: Bug 1267434 - Unnecessary whitespace displayed in about:privatebrowing; r?Gijs

(In reply to Francesco Lodolo [:flod] from comment #9)
> I'm a bit torn, but I think we should try to land this as soon as possible
> in aurora.


Approval Request Comment
[Feature/regressing bug #]: bug 1259340
[User impact if declined]: l10n issues for non-English
[Describe test coverage new/current, TreeHerder]: nope
[Risks and why]: very low, tiny string + HTML change
[String/UUID change made/needed]: yes, there is a string change, but see above - we branched 2 days ago, so IMO we should just get it in real quick. :-\
Flags: needinfo?(sledru)
Flags: needinfo?(lhenry)
Attachment #8745866 - Flags: approval-mozilla-aurora?

Comment 12

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/099e2bf3ce03
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Iteration: --- → 49.1 - May 9
Flags: qe-verify?
Priority: P3 → P1
Comment on attachment 8745866 [details]
MozReview Request: Bug 1267434 - Unnecessary whitespace displayed in about:privatebrowing; r?Gijs

OK to uplift to aurora though there is a small string change, since flod approves
Flags: needinfo?(lhenry)
Attachment #8745866 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/7c7a6fe3b94b
status-firefox48: --- → fixed

Updated

a year ago
Flags: needinfo?(sledru)
You need to log in before you can comment on or make changes to this bug.