Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

English locale will now show a smaller version of AM/PM #5520

Merged
merged 1 commit into from
Nov 10, 2016
Merged

English locale will now show a smaller version of AM/PM #5520

merged 1 commit into from
Nov 10, 2016

Conversation

bsclifton
Copy link
Member

@bsclifton bsclifton commented Nov 9, 2016

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

NOTE: Ready for merge after 0.12.9 moves to dev-channel 😄

Fixes #5381

Auditors: @cezaraugusto

Test Plan

  1. Set your locale in Preferences to be English (U.S.)
  2. Visit about:newtab; AM/PM (if shown for you) should be smaller than the font-size used for the hour/minute

Screenshots

Japanese (24 hour time; no AM/PM shown)
screen shot 2016-11-09 at 2 49 06 pm

English (US; shows AM/PM smaller, vertically aligned to top)
screen shot 2016-11-09 at 2 50 43 pm

Fixes #5381

Auditors: @cezaraugusto

Test Plan:
1. Set your locale in Preferences to be English (U.S.)
2. Visit about:newtab; AM/PM (if shown for you) should be smaller than the font-size used for the hour/minute
@cezaraugusto
Copy link
Contributor

++

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants