Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 3 additions & 11 deletions src/components/views/Footer.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -49,17 +49,9 @@ export default class extends React.Component {
<h3>Firefox</h3>
<nav>
<ul>
<li><a href="https://www.firefox.com/">Download Firefox Web browser</a></li>
<li>
<a href="https://www.firefox.com/browsers/desktop/">
Desktop Browser for Mac, Window, Linux
</a>
</li>
<li>
<a href="https://www.firefox.com/download/android/">
Mobile Browser for Android
</a>
</li>
<li><a href="https://www.firefox.com/download/">Download Firefox Web browser</a></li>
<li><a href="https://www.firefox.com/">Desktop Browser for Mac, Window, Linux</a></li>
<li><a href="https://www.firefox.com/download/android/">Mobile Browser for Android</a></li>
Comment on lines -52 to +54

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is mostly whitespace for consistency, however the material change hiding in this diff is:
To get rid of the no longer maintained desktop category page I've swapped download and home so that it makes better sense and can use the modern pages' links that way.

<li><a href="https://www.firefox.com/download/ios/">Mobile Browser for iOS</a></li>
<li className="footer-social">
<ul>
Expand Down
4 changes: 2 additions & 2 deletions src/components/views/Home.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -82,9 +82,9 @@ export default () => {
You can learn more about this report by reading
{' '}<a href="https://blog.mozilla.org/blog/2018/08/28/lets-be-transparent/">our announcement on the Mozilla blog</a>
{' '}and by exploring the projects that power it:
{' '}<a href="https://github.com/mozilla/Fx_Usage_Report">FX_Usage_Report</a> (data processing and documentation),
{' '}<a href="https://github.com/mozilla/firefox-public-data-report-etl">Firefox Public Data Report ETL</a> (data processing and documentation),
{' '}<a href="https://github.com/mozilla/ensemble-transposer">ensemble-transposer</a> (formatting and metadata), and
{' '}<a href="https://github.com/mozilla/ensemble">ensemble</a> (data visualization).
{' '}<a href="https://github.com/mozilla/ensemble">ensemble</a> (data visualization).
Comment on lines -85 to +87

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to have fallen off the wagon to apply from the previous review.
(+reverting whitespace from last PR)

</p>
</section>
</article>
Expand Down
Loading