-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates from bedrock #404
Merged
Merged
Updates from bedrock #404
Changes from all commits
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,20 +23,45 @@ home-product-firefox-browsing = Private & secure browsing | |
home-cta-get-firefox = Get { -brand-name-firefox } | ||
home-product-pocket-articles = The web’s most intriguing articles | ||
home-cta-get-pocket = Get { -brand-name-pocket} | ||
|
||
# Obsolete string (expires: 2024-07-16) | ||
home-product-relay-masks = Easy-to-use email & phone masks | ||
|
||
home-product-relay-hide-your = Hide your phone number and email from spammers | ||
home-cta-get-relay = Get { -brand-name-relay } | ||
home-product-vpn-trust = A VPN you can trust | ||
home-cta-get-vpn = Get { -brand-name-mozilla-vpn } | ||
|
||
# Obsolete string (expires 2024-07-23) | ||
home-product-monitor-data = Data breach alerts | ||
|
||
home-product-monitor-protect-your = Protect your private info from data brokers | ||
home-cta-get-monitor = Get { -brand-name-monitor } | ||
home-product-fakespot-detect = Detect fake shopping reviews | ||
home-cta-get-fakespot = Get { -brand-name-fakespot } | ||
|
||
home-mozilla-takes-bets = “{ -brand-name-mozilla } is taking bets to show the world there’s a business to be made with trustworthy AI. That includes putting things like human rights, data protection and transparency at the core of how these complex systems work.” | ||
|
||
# Politico is a brand name for a news organization | ||
home-politico-cite = Politico | ||
|
||
home-join-us-in-shaping = Join us in shaping trustworthy AI | ||
home-work-on-ai = { -brand-name-mozilla }’s work with AI isn’t just a new thing—we’ve spent years funding, building and advocating for AI that’s open, fair and developed responsibly. Our focus is on creating AI that serves the people, prioritizes transparency and supports the public good, not corporate agendas. | ||
home-read-more = Read more | ||
|
||
# Obsolete string (expires: 2024-07-16) | ||
home-is-mozilla-a-corporation = Is { -brand-name-mozilla } a corporation or a non-profit? Actually, both. | ||
|
||
# Obsolete string (expires: 2024-07-16) | ||
home-mozilla-consists-of = { -brand-name-mozilla } consists of two organizations. The { -brand-name-mozilla-corporation } is wholly owned by the non-profit 501(c) { -brand-name-mozilla-foundation }. Which means we aren’t beholden to any shareholders — only to our mission. | ||
|
||
home-so-what-is-mozilla = So, what is { -brand-name-mozilla }? | ||
|
||
# Variables | ||
# $ventures - link to https://mozilla.vc/ | ||
# $mozai - link to https://mozilla.ai/ | ||
home-at-its-core = At its core, { -brand-name-mozilla } is an activist organization led by the { -brand-name-mozilla-foundation } that makes change in the world through a variety of ventures including { -brand-name-mozilla-corporation }, MZLA, <a {$ventures}>{ -brand-name-mozilla-ventures }</a> and <a {$mozai}>{ -brand-name-mozilla-ai }</a>. How are we different? Because we’re mission-driven, it means we have the freedom to make all of our decisions based on what’s best for the internet and for everyone online, not based on the demands of shareholders — we don’t actually have any of those. | ||
|
||
home-learn-about-mofo = Learn about the { -brand-name-mozilla-foundation } | ||
|
||
home-join-the-fight = Join the fight for a healthy internet | ||
|
@@ -57,3 +82,8 @@ home-featured-product = Featured product | |
|
||
# HTML for visual formatting. "Blur" here is used as a metaphor for hiding or obscuring something. | ||
home-feature-blur-your-location = Blur your location & activity using <span>{ -brand-name-mozilla-vpn }</span> | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. remove an extra line break |
||
# Mozilla Foundation donation promo | ||
home-mofo-build-our-movement = Build our movement | ||
home-mofo-donate-to-mofo-today = Donate to { -brand-name-mozilla-foundation } today to ensure the internet remains open and accessible to all. | ||
home-mofo-donate = Donate |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use M dash.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This:
already is
U+2014
EM DASH@peiying2 Do you mean another occurence?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The only inconsistent thing here is that while this sentence uses the dash correctly — without the spaces on both sides (at it should be IIRC), the other string on L64 has spaces around it. So the only thing I see here wrt dashes would be making these two occurrences more consistent: mozilla/bedrock#14602 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OTOH the "correct" (non)spacing produces some ugly line breaks at times:
so the question is whether to give up and just have spaces around it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've asked the copy writing team for clarification on spaces and em dashes. It seems we are doing it a couple different ways currently and since we are trying to go a little more colloquial with our tone, that might be true of our grammar too so I don't feel confident making a call myself.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, it was hard to see it. I agree with @janbrasna that there should be a space before and after the M dash, but it has been inconsistent throughout. If there is a branding or style guide somewhere, let me know. For now, I will take this as is and move forward.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I talked to someone in brand and they said spaces around the dash are the way to go.
They also said they don't have a style guide.