Closed Bug 876191 Opened 7 years ago Closed 7 years ago

Defect - Two spaces between "the" & "nightly" under the "About" flyout

Categories

(Firefox for Metro Graveyard :: Flyouts, defect, P2)

x86_64
Windows 8.1
defect

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 24

People

(Reporter: kjozwiak, Assigned: bbondy)

References

Details

(Whiteboard: [shovel-ready] feature=Defect c=Settings_pane_options_and_about u=metro_firefox_user p=1)

Attachments

(2 files)

This is pretty trivial but there are two spaces between "the" & "nightly" under the "About" flyout. Its not consistent and it looks a little off when you see two spaces between the two words instead of one.

I also think that "nightly" should be changed to "Nightly" to keep it consistent throughout the flyouts.

I attached a screenshot to illustrate the two spaces. (the gap is wider between "the" & "nightly" compared to the rest of the sentence)

Steps to reproduce the issue:

1) Open Firefox Metro
2) Swipe the "App Bar" and select "Settings" and then "About"

Current Behavior:

- You will notice a wider gab between "the" & "nightly"

Expected Behavior:

- Gap should be the same size as the others, would look nicer if "nightly" was changed to "Nightly"
Priority: -- → P2
Whiteboard: feature=Defect c=Settings_pane_options_and_about u=metro_firefox_user p=0 → [shovel-ready] feature=Defect c=Settings_pane_options_and_about u=metro_firefox_user p=0
p=1
Attached patch Patch v1.Splinter Review
This just copies the same desktop firefox style for #currentChannel
Assignee: nobody → netzen
Status: NEW → ASSIGNED
Attachment #755140 - Flags: review?(ally)
Blocks: metrov1it8
No longer blocks: metrov1defect&change
Whiteboard: [shovel-ready] feature=Defect c=Settings_pane_options_and_about u=metro_firefox_user p=0 → [shovel-ready] feature=Defect c=Settings_pane_options_and_about u=metro_firefox_user p=1
QA Contact: jbecerra
Comment on attachment 755140 [details] [diff] [review]
Patch v1.

Review of attachment 755140 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm.

Kamil, I know the lower case 'nightly' is incorrect grammar, but I notice that on my desktop aurora has the same issue: "You are currently on the aurora update channel". For now, it is parity, and I'd bet it has something to do with localization.
Attachment #755140 - Flags: review?(ally) → review+
https://hg.mozilla.org/mozilla-central/rev/666e862aaac3
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
For testing and verification.
Flags: needinfo?(kamiljoz)
Went through the following "Defect" for Testing & Verification without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-06-01-03-09-38-mozilla-central/

- Ensured that the two spaces have been removed and the formatting looks correct throughout the "About" flyout
- Ensured that the "nightly" is not capitalized as per Comment 3 (it's in parity with other release channels as Ally mentioned)
- Ensured that the "About" flyout worked without any issues
- Ensured that the "Read the Nightly privacy online" link worked without any issues
- Ensured that the above test cases also work in Filled View
Status: RESOLVED → VERIFIED
Flags: needinfo?(kamiljoz)
Went through the following "Defect" for iteration #8 testing without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-06-10-03-11-47-mozilla-central/

- Went through the original test cases from Comment 0 without any issues
- Went through the test cases from Comment 7 without any issues
Went through the following "Defect" for iteration #9 testing without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-06-25-03-12-38-mozilla-central/

- Went through the original test case added in comment 0 without any issues
- Went through the test cases from comment 7 without any issues
- Ensured that the correct version # & date is being listed under the "About" flyout
Blocks: 831958
Went through the following "Defect" for iteration #10 testing without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-07-19-03-02-04-mozilla-central/

- Went through the original test case added in comment 0 without any issues
- Went through the test cases from comment 7 without any issues
- Ensured that the correct version # & date is being listed under the "About" flyout
Went through the following "Defect" for iteration #12 testing without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-08-20-03-02-06-mozilla-central/

- Went through the original test case added in comment 0 without any issues
- Went through the test cases from comment 7 without any issues
- Ensured that the correct version # & date is being listed under the "About" flyout
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.