Skip to content

Commit

Permalink
Review fixes
Browse files Browse the repository at this point in the history
* Misc fixes to strings and Fluent files.
* Remove third-party brand variables.
* Deleting obsolete CSS.
* Fix PiP JS
* Add image borders.
  • Loading branch information
craigcook committed Aug 17, 2023
1 parent 6f50eb2 commit e682852
Show file tree
Hide file tree
Showing 30 changed files with 112 additions and 427 deletions.
4 changes: 2 additions & 2 deletions bedrock/firefox/templates/firefox/features/adblocker.html
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ <h2>{{ftl('features-adblocker-create-a-tracker-free') }}</h2>
</figure>

<h2>{{ftl('features-adblocker-choose-your-level-of-protection') }}</h2>
<p>{{ ftl('features-adblocker-to-start-click-on-the') }}</p>
<p>{{ ftl('features-adblocker-to-start-click-on-the-v2', fallback='features-adblocker-to-start-click-on-the') }}</p>

<h2>{{ftl('features-adblocker-go-easy-with-standard') }}</h2>
<p>{{ ftl('features-adblocker-if-ads-dont-bother-you', url=url('firefox.features.private-browsing')) }}</p>
Expand Down Expand Up @@ -127,5 +127,5 @@ <h2>{{ftl('features-adblocker-send-a-do-not-track-signal') }}</h2>

<h2>{{ ftl('features-adblocker-speed-up-thanks-to-ad') }}</h2>
<p>{{ ftl('features-adblocker-in-some-cases-an-ad-blocker') }}</p>
<p>{{ ftl('features-adblocker-if-you-want-to-learn-more', url=url('firefox.new')) }}</p>
<p>{{ ftl('features-adblocker-if-you-want-to-learn-more-v2', fallback='features-adblocker-if-you-want-to-learn-more', url=url('firefox.new')) }}</p>
{% endblock %}
4 changes: 2 additions & 2 deletions bedrock/firefox/templates/firefox/features/base-article.html
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

{% block page_css %}
{{ css_bundle('protocol-split') }}
{{ css_bundle('firefox-features-new-article') }}
{{ css_bundle('firefox-features-article') }}
{% endblock page_css %}

{% block article_title_short %}{% endblock %}
Expand Down Expand Up @@ -65,5 +65,5 @@ <h3>{{ ftl('features-shared-footer-cta-title') }}</h3>
{% endblock content %}

{% block js %}
{{ js_bundle('firefox_features_new_article') }}
{{ js_bundle('firefox_features_article') }}
{% endblock %}
6 changes: 3 additions & 3 deletions bedrock/firefox/templates/firefox/features/customize.html
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@
trending='href="https://addons.mozilla.org/firefox/search/?sort=hotness&type=statictheme&utm_source=www.mozilla.org&utm_medium=referral&utm_campaign=firefox-features" rel="external noopener"',
recommended='href="https://addons.mozilla.org/firefox/search/?promoted=recommended&sort=random&type=statictheme&utm_source=www.mozilla.org&utm_medium=referral&utm_campaign=firefox-features" rel="external noopener"',
music='href="https://addons.mozilla.org/firefox/themes/category/music/?utm_source=www.mozilla.org&utm_medium=referral&utm_campaign=firefox-features" rel="external noopener"',
seasonal='href="https://addons.mozilla.org/firefox/themes/category/seasonal/?href="utm_source=www.mozilla.org&utm_medium=referral&utm_campaign=firefox-features" rel="external noopener"',
sports='href="https://addons.mozilla.org/firefox/themes/category/sports/?href="utm_source=www.mozilla.org&utm_medium=referral&utm_campaign=firefox-features" rel="external noopener"',
nature='href="https://addons.mozilla.org/firefox/themes/category/nature/?href="utm_source=www.mozilla.org&utm_medium=referral&utm_campaign=firefox-features" rel="external noopener"'
seasonal='href="https://addons.mozilla.org/firefox/themes/category/seasonal/?utm_source=www.mozilla.org&utm_medium=referral&utm_campaign=firefox-features" rel="external noopener"',
sports='href="https://addons.mozilla.org/firefox/themes/category/sports/?utm_source=www.mozilla.org&utm_medium=referral&utm_campaign=firefox-features" rel="external noopener"',
nature='href="https://addons.mozilla.org/firefox/themes/category/nature/?utm_source=www.mozilla.org&utm_medium=referral&utm_campaign=firefox-features" rel="external noopener"'
) }}
</p>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

{% block page_desc %}{{ ftl('features-fingerprinting-fingerprinting-is-a-desc') }}{% endblock %}

{% block article_title_short %}Fingerprinter blocking{% endblock %}
{% block article_title_short %}{{ ftl('features-fingerprinting-fingerprint-blocking') }}{% endblock %}
{% block article_title %}{{ ftl('features-fingerprinting-firefox-blocks-fingerprinting') }}{% endblock %}

{% block article_content %}
Expand Down
6 changes: 3 additions & 3 deletions bedrock/firefox/templates/firefox/features/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ <h3 class="c-feature-item-title">{{ title }}</h3>
{%- endmacro %}

{% block page_css %}
{{ css_bundle('firefox-features-new-index')}}
{{ css_bundle('firefox-features-index')}}
{% endblock page_css %}

{% block page_title %}{{ ftl('features-index-firefox-browser-features') }}{% endblock page_title %}
Expand Down Expand Up @@ -144,8 +144,8 @@ <h3 class="c-feature-item-title">{{ title }}</h3>

{{ feature_item(
link=url('firefox.features.fingerprinting'),
title=ftl('features-index-fingerprinter-blocking'),
ga_title="Fingerprinter blocking",
title=ftl('features-index-fingerprint-blocking'),
ga_title="Fingerprint blocking",
desc=ftl('features-index-ditch-the-sticky-ads'),
) }}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -89,5 +89,6 @@ <h3>{{ ftl('features-pip-3-more-ways-to-use-picture', fallback='features-pip-3-w
{% endblock article_content %}

{% block js %}
{{ super() }}
{{ js_bundle('firefox-features-pip') }}
{% endblock %}
10 changes: 5 additions & 5 deletions bedrock/firefox/urls.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,15 +37,15 @@
page("firefox/developer/", "firefox/developer/index.html", ftl_files=["firefox/developer"]),
page("firefox/enterprise/", "firefox/enterprise/index.html", ftl_files=["firefox/enterprise"]),
page("firefox/facebookcontainer/", "firefox/facebookcontainer/index.html", ftl_files=["firefox/facebook_container"]),
page("firefox/features/", "firefox/features/index.html", ftl_files=["firefox/features/index", "firefox/features/shared"]),
page("firefox/features/", "firefox/features/index.html", ftl_files=["firefox/features/index-2023", "firefox/features/shared"]),
page("firefox/features/customize/", "firefox/features/customize.html", ftl_files=["firefox/features/customize", "firefox/features/shared"]),
page("firefox/features/add-ons/", "firefox/features/add-ons.html", ftl_files=["firefox/features/add-ons", "firefox/features/shared"]),
page("firefox/features/pinned-tabs/", "firefox/features/pinned-tabs.html", ftl_files=["firefox/features/pinned-tabs", "firefox/features/shared"]),
page("firefox/features/eyedropper/", "firefox/features/eyedropper.html", ftl_files=["firefox/features/eyedropper", "firefox/features/shared"]),
page("firefox/features/pdf-editor/", "firefox/features/pdf-editor.html", ftl_files=["firefox/features/pdf-editor", "firefox/features/shared"]),
page("firefox/features/adblocker/", "firefox/features/adblocker.html", ftl_files=["firefox/features/adblocker", "firefox/features/shared"]),
page("firefox/features/bookmarks/", "firefox/features/bookmarks.html", ftl_files=["firefox/features/bookmarks", "firefox/features/shared"]),
page("firefox/features/fast/", "firefox/features/fast.html", ftl_files=["firefox/features/fast", "firefox/features/shared"]),
page("firefox/features/bookmarks/", "firefox/features/bookmarks.html", ftl_files=["firefox/features/bookmarks-2023", "firefox/features/shared"]),
page("firefox/features/fast/", "firefox/features/fast.html", ftl_files=["firefox/features/fast-2023", "firefox/features/shared"]),
page(
"firefox/features/block-fingerprinting/",
"firefox/features/fingerprinting.html",
Expand All @@ -54,7 +54,7 @@
page(
"firefox/features/password-manager/",
"firefox/features/password-manager.html",
ftl_files=["firefox/features/password-manager", "firefox/features/shared"],
ftl_files=["firefox/features/password-manager-2023", "firefox/features/shared"],
),
page(
"firefox/features/private/",
Expand All @@ -64,7 +64,7 @@
page(
"firefox/features/private-browsing/",
"firefox/features/private-browsing.html",
ftl_files=["firefox/features/private-browsing", "firefox/features/shared"],
ftl_files=["firefox/features/private-browsing-2023", "firefox/features/shared"],
),
page("firefox/features/sync/", "firefox/features/sync.html", ftl_files=["firefox/features/sync", "firefox/features/shared"]),
path("firefox/features/translate/", views.firefox_features_translate, name="firefox.features.translate"),
Expand Down
10 changes: 8 additions & 2 deletions l10n/en/firefox/features/adblocker.ftl
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,10 @@ features-adblocker-create-a-tracker-free = Create a tracker-free zone with Conte
features-adblocker-on-firefox-you-can-use = On { -brand-name-firefox }, you can use <a href="{ $privacy }">Privacy</a> or <a href="{ $blocking }">Content Blocking</a> settings to get even more control over ad trackers that serve you the ads.
features-adblocker-choose-your-level-of-protection = Choose your level of protection
features-adblocker-to-start-click-on-the = To start, click the shield to the left of the address bar on any webpage and select “Protection Settings.” This will open the { -brand-name-firefox } Settings Privacy & Security panel in a new tab. You should see a blue pop-up with different selections.
features-adblocker-to-start-click-on-the-v2 = To start, click the shield to the left of the address bar on any webpage and select “Protection Settings.” This will open the { -brand-name-firefox } Settings Privacy & Security panel in a new tab. You should see a blue pop-up with different selections.
# Obsolete string
features-adblocker-to-start-click-on-the = To start, click on the { -brand-name-firefox } menu in the top right-hand corner of your screen. It looks like three lines stacked on top of each other. In the drop-down menu, click on Content Blocking. You should see a blue pop-up with different selections.
features-adblocker-go-easy-with-standard = Go easy with Standard mode
# Variables:
Expand Down Expand Up @@ -69,4 +72,7 @@ features-adblocker-in-some-cases-an-ad-blocker = In some cases, an ad blocker ca
# Variables:
# $url (url) - link to https://www.mozilla.org/firefox/new/
features-adblocker-if-you-want-to-learn-more = If you want to learn more about ad blocking, there are hundreds of ad blocker extensions available for { -brand-name-firefox } and other browsers. If you want to try out the ad blockers { -brand-name-firefox } uses, <a href="{ $url }">click here to download</a> a browser that puts privacy first.
features-adblocker-if-you-want-to-learn-more-v2 = If you want to learn more about ad blocking, there are hundreds of ad blocker extensions available for { -brand-name-firefox } and other browsers. If you want to try out the ad blockers { -brand-name-firefox } uses, <a href="{ $url }">click here to download</a> a browser that puts privacy first.
# Obsolete string
features-adblocker-if-you-want-to-learn-more = If you want to learn more about ad blocking, there are hundreds of ad blocker extensions available for { -brand-name-firefox } and other browsers. If want to try out the ad blockers { -brand-name-firefox } uses, <a href="{ $url }">click here to download</a> a browser that puts privacy first.
Original file line number Diff line number Diff line change
Expand Up @@ -14,4 +14,4 @@ features-add-ons-extensions-also-known-as-add-ons = Extensions – also known as
# $language (url) - link to https://addons.mozilla.org/firefox/addon/languagetool/
# $customize (url) - link to https://www.mozilla.org/firefox/features/customize/
# $amo (url) - link to https://addons.mozilla.org
features-add-ons-there-are-firefox-add-ons-that = There are { -brand-name-firefox } add-ons that <a { $fbcontainer }>stop { -brand-name-facebook } from tracking you around the web</a>, <a { $translate }>translate text into other languages</a>, <a { $language }>check your spelling or grammar</a>, or <a { $customize }>spruce up the way your browser looks</a>. You’ll find these and thousands of other free extensions at <a { $amo }>addons.mozilla.org</a>.
features-add-ons-there-are-firefox-add-ons-that = There are { -brand-name-firefox } add-ons that <a { $fbcontainer }>stop Facebook from tracking you around the web</a>, <a { $translate }>translate text into other languages</a>, <a { $language }>check your spelling or grammar</a>, or <a { $customize }>spruce up the way your browser looks</a>. You’ll find these and thousands of other free extensions at <a { $amo }>addons.mozilla.org</a>.
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ bookmark-manager-collect-your-bookmarks-in-folders = Collect your bookmarks in f
# Used as an accessible text alternative for an image
bookmark-manager-image-of-the-bookmark-manager = Image of the bookmark manager window in { -brand-name-firefox }.
bookmark-manager-easily-import-bookmarks = Easily import bookmarks
bookmark-manager-you-can-import-your-bookmarks = You can import your bookmarks from { -brand-name-chrome }, { -brand-name-safari } or { -brand-name-edge } with { -brand-name-firefox }’s import wizard. Just click Bookmarks > Manage Bookmarks and then select “Import and Backup”.
bookmark-manager-you-can-import-your-bookmarks = You can import your bookmarks from Chrome, Safari or Edge with { -brand-name-firefox }’s import wizard. Just click Bookmarks > Manage Bookmarks and then select “Import and Backup”.
# Used as an accessible text alternative for an image
bookmark-manager-image-of-the-firefox-import-wizard = Image of the { -brand-name-firefox } import wizard dialog, showing options to import settings and data from other browsers.
Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
features-fast-is-firefox-the-fastest-browser = Is { -brand-name-firefox } the fastest browser?
# HTML description
features-fast-firefox-uses-less-memory-than = { -brand-name-firefox } uses less memory than { -brand-name-chrome }, so your other programs can keep running at top speed.
features-fast-firefox-uses-less-memory-than = { -brand-name-firefox } uses less memory than Chrome, so your other programs can keep running at top speed.
features-fast-browse-faster = Browse faster
# The odd punctuation here is meant to represent how a computer can run slow.
Expand Down
1 change: 1 addition & 0 deletions l10n/en/firefox/features/fingerprinting.ftl
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ features-fingerprinting-what-is-fingerprinting-title = What is fingerprinting an
# page description
features-fingerprinting-fingerprinting-is-a-desc = Fingerprinting is a type of online tracking that’s more invasive than ordinary cookie-based tracking — that’s why { -brand-name-firefox-browser } blocks it.
features-fingerprinting-fingerprint-blocking = Fingerprint blocking
features-fingerprinting-firefox-blocks-fingerprinting = { -brand-name-firefox } blocks fingerprinting
features-fingerprinting-ditch-the-sticky = Ditch the sticky ads following you around with { -brand-name-firefox } browser’s built-in fingerprinting blockers.
Expand Down
27 changes: 0 additions & 27 deletions l10n/en/firefox/features/independent.ftl

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ features-index-firefox-browser-features = { -brand-name-firefox } browser featur
features-index-firefox-is-the-fast-lightweight = { -brand-name-firefox } is the fast, lightweight, privacy-focused browser that works across all your devices.
features-index-is-firefox-a-fast-browser = Is { -brand-name-firefox } a fast browser?
features-index-firefox-uses-less-memory-than = { -brand-name-firefox } uses less memory than { -brand-name-chrome }, so your other programs can keep running at top speed.
features-index-firefox-uses-less-memory-than = { -brand-name-firefox } uses less memory than Chrome, so your other programs can keep running at top speed.
features-index-is-firefox-a-private-browser = Is { -brand-name-firefox } a private browser?
features-index-were-focused-on-your-right-to = We’re focused on your right to privacy. Your data, your web activity, your life online is protected with { -brand-name-firefox }.
features-index-free-password-manager = Free password manager
Expand All @@ -32,7 +32,7 @@ features-index-eyedropper-tool = Eyedropper tool
features-index-identify-the-exact-color-on-a = Identify the exact color on a page and copy its hex code.
features-index-pdf-editor = PDF editor
features-index-view-and-edit-pdf-files = View and edit PDF files right in { -brand-name-firefox }.
features-index-fingerprinter-blocking = Fingerprinter blocking
features-index-fingerprint-blocking = Fingerprint blocking
features-index-ad-tracker-blocking = Ad tracker blocking
features-index-firefox-automatically-blocks = { -brand-name-firefox } automatically blocks 2000+ ad trackers from following you around the internet.
Expand Down
Loading

0 comments on commit e682852

Please sign in to comment.