Closed
Bug 1447972
Opened 7 years ago
Closed 7 years ago
Missing full stop period at end of "Choose what you see when… and new tabs"
Categories
(Firefox :: New Tab Page, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox59 | --- | unaffected |
firefox60 | --- | unaffected |
firefox61 | --- | fixed |
People
(Reporter: winson.wen1, Assigned: rrosario)
References
(Blocks 1 open bug)
Details
(Whiteboard: [strings m-c landed])
Attachments
(2 files)
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:61.0) Gecko/20100101 Firefox/61.0
Build ID: 20180321220044
Steps to reproduce:
Open about:preferences
Click on the Home tab
Actual results:
Drop-down menus are different lengths
"(Default)" text in drop-down menus aren't grey/faded
No colon after "Homepage and new windows" and "New tabs"
Expected results:
Drop-down menus are the same length
"(Default)" text in drop-down menus is grey/faded
Colon after "Homepage and new windows" and "New tabs"
As per https://mozilla.invisionapp.com/share/EGFI16P8Y36#/screens/280785532
Reporter | ||
Comment 1•7 years ago
|
||
Also full-stop missing at the end of "Choose what you see when you open your homepage, new windows, and new tabs"
Reporter | ||
Comment 2•7 years ago
|
||
Also, gap between header ans description should be consistent.
Sorry for spam, forgot to include this in the original comment
Updated•7 years ago
|
Blocks: 1417155
Has Regression Range: --- → irrelevant
Has STR: --- → yes
Component: Untriaged → Activity Streams: Newtab
Updated•7 years ago
|
Status: UNCONFIRMED → NEW
status-firefox59:
--- → unaffected
status-firefox60:
--- → unaffected
status-firefox61:
--- → affected
Ever confirmed: true
OS: Unspecified → All
Hardware: Unspecified → All
Comment 3•7 years ago
|
||
The colon was removed to match the rest of about:preferences as per 1417155 comment 36.
Bug 1447988 covers the width.
Design is good with non-gray "(Default)" if it's not supported easily, which seems like it would require a new xbl binding to have double <xul:label>s differently styled: https://searchfox.org/mozilla-central/source/toolkit/content/widgets/menu.xml#180-187
So the remaining issue is the full stop.
k88hudson, do you know if it was intentionally omitted?
Flags: needinfo?(khudson)
Summary: Home preferences don't match spec → Missing full stop period at end of "Choose what you see when… and new tabs"
Comment 4•7 years ago
|
||
The full stop was not intentionally omitted – it should probably be consistent at least across our other sections in about:preferences#home. I think this should be fixed
Flags: needinfo?(khudson)
Comment 5•7 years ago
|
||
flod, this punctuation addition should require an id change to trigger relocalization?
Flags: needinfo?(francesco.lodolo)
Whiteboard: [strings m-c needed]
Comment 6•7 years ago
|
||
(In reply to Ed Lee :Mardak from comment #5)
> flod, this punctuation addition should require an id change to trigger
> relocalization?
Yes, if you want to ensure that all locales notice the change and adapt their translation, you need a new ID. For punctuation, most locales follow English, so it's good to change the ID.
Flags: needinfo?(francesco.lodolo)
Updated•7 years ago
|
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → rrosario
Comment hidden (mozreview-request) |
Comment 8•7 years ago
|
||
Comment on attachment 8964670 [details]
Bug 1447972 - Missing full stop period in home page preferences
gandalf, any differences with ftl for string changes?
Attachment #8964670 -
Flags: review?(edilee) → review?(gandalf)
Comment 9•7 years ago
|
||
We are working toward a bit more sophisticated model which will only require ID change when semantics of the message changes.
In the new model we may be able to distinguish stylistic change from en-US specific spelling/punctuation change, which would be helpful here.
My gut feeling is that it is ok to leave the ID, but I'll NI flod for confirmation.
Flags: needinfo?(francesco.lodolo)
Comment 10•7 years ago
|
||
We need the new ID if we want consistency in the UI.
Most locales follow blindly en-US when it comes to punctuation. Result: English is inconsistent, only 8 locales don't have the issue (consciously or not)
https://transvision.mozfr.org/string/?entity=browser/browser/preferences/preferences.ftl:home-new-windows-tabs-description&repo=gecko_strings
Flags: needinfo?(francesco.lodolo)
Comment 11•7 years ago
|
||
mozreview-review |
Comment on attachment 8964670 [details]
Bug 1447972 - Missing full stop period in home page preferences
https://reviewboard.mozilla.org/r/233360/#review239114
Ok, this is a great example of the planned string versioning feature then. Until that gets enabled, let's keep changing the ID even in such cases (my heart bleeds a little) :(
Attachment #8964670 -
Flags: review?(gandalf) → review+
Comment 12•7 years ago
|
||
Pushed by edilee@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/1d7c8c50aeab
Missing full stop period in home page preferences r=gandalf
Comment 13•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Updated•7 years ago
|
Whiteboard: [strings m-c needed] → [strings m-c landed]
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•