Updates from bedrock #431
Updates from bedrock #431
Conversation
From file changes in mozilla/bedrock@cf21cf2
There was a problem hiding this comment.
@craigcook not sure who is working on this PR, but in looking at https://www-dev.allizom.org/en-US/ - the title case or capitalization of title case is inconsistent depending on where you look. In most situation, it looks like this Title case. But some are not: Firefox Blog (under Firefox Browsers). Under About us where Innovation Projects is not consistent with the rest (MoFo is an exception in this case).
The footer's title case seems to have capital initials like Footer Title Case vs. Navigation title case (or inconsistent within each menu).
en/navigation_refresh.ftl
Outdated
| navigation-refresh-mozilla-manifesto = { -brand-name-mozilla } manifesto | ||
| navigation-refresh-mozilla-foundation = { -brand-name-mozilla-foundation } | ||
| navigation-refresh-get-involved = Get involved | ||
| navigation-refresh-innovation-projects = Innovation Projects |
There was a problem hiding this comment.
Should this be Innovation projects (lower case P) to be consistent with other list of items?
There was a problem hiding this comment.
Sentance case is the official word going foward.
There was a problem hiding this comment.
People are doing a bit of everything right now. @craigcook do you want take a pass on capitalization on Monday? I can if you're busy but I know you're better with speling.
There was a problem hiding this comment.
Let's get this updated soon as it's time sensitive. Anymore strings to come in the next couple of days?
There was a problem hiding this comment.
@alexgibson not sure if you can help fix this. We are pressing for time. If this is the last update before next week, let's finalize it. If there are more things to come, can we bundle them? Thank you!
There was a problem hiding this comment.
@peiying2 I will take a look here thanks for the ping
Edit: I'm going to assume sentence case is what we should be using for all titles in the nav
There was a problem hiding this comment.
@peiying2 PR here that should hopefully address things: mozilla/bedrock#15164
en/navigation_refresh.ftl
Outdated
| navigation-refresh-firefox-focus = { -brand-name-firefox-focus } | ||
| navigation-refresh-firefox-relay = { -brand-name-firefox-relay } | ||
| navigation-refresh-all-products = All products | ||
| navigation-refresh-firefox-blog = { -brand-name-firefox } Blog |
There was a problem hiding this comment.
Should this be lower case B so it reads Firefox blog?
From file changes in mozilla/bedrock@2702fb9
From file changes in mozilla/bedrock@cf21cf2