Skip to content
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

Firefox sidebar changes #5672

Merged
merged 1 commit into from
Mar 18, 2022
Merged

Firefox sidebar changes #5672

merged 1 commit into from
Mar 18, 2022

Conversation

dipikabh
Copy link
Contributor

@dipikabh dipikabh commented Mar 18, 2022

Summary

Fixes #5096.

Thanks @schalkneethling for your time in helping me make these changes!

Problem

Non-working links under Firefox internals and Building and contributing

Solution

As suggested in the issue, replaced the two menus Firefox internals and Building and contributing with Firefox documentation (see After screenshot), which now links to https://firefox-source-docs.mozilla.org/.

In addition, the following changes have been made:

  • Renamed Firefox developer release notes (see Before screenshot) to Firefox releases (see After screenshot)
  • Updated text for Firefox developer release notes to Firefox release notes for developers [title on landing page needs to be updated. I'll submit a separate PR on mdn/content.] - Update title of FF release landing page content#14010
  • Added a submenu to link to Experimental features in Firefox (see After screenshot)

Replicated all the above changes in other language sections (which will require translation).

Screenshots

Before

Screenshot 2022-03-18 at 10 09 17 AM

After

Screenshot 2022-03-18 at 12 24 07 PM


How did you test this change?

  • Checked the changes on local build through yarn dev.
  • Updated firefoxsidebar.test.js file with Schalk's help so that all tests pass.

* Remove Firefox internals menu
* Remove Building and contributing menu
* Add link to Experimental features
* Update Firefox developer release notes text
* Make the above changes for all languages
@dipikabh dipikabh changed the title Firefox sidebar changes (#5096) Firefox sidebar changes Mar 18, 2022
@Rumyra
Copy link
Contributor

Rumyra commented Mar 18, 2022

Looks great to me @dipikabh 🙌 - I'll let @schalkneethling merge tho as he'll be more familiar with the changes :)

@schalkneethling schalkneethling added the macros tracking issues related to kumascript macros label Mar 18, 2022
Copy link
Contributor

@schalkneethling schalkneethling left a comment

Choose a reason for hiding this comment

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

Thanks, @dipikabh 🎉

@schalkneethling
Copy link
Contributor

@dipikabh, for future reference, pull requests to Yari need commits to be signed. See here for more info: https://docs.github.com/articles/about-gpg/

@schalkneethling schalkneethling self-assigned this Mar 18, 2022
@schalkneethling schalkneethling merged commit a0bee3c into mdn:main Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
macros tracking issues related to kumascript macros
Projects
Development

Successfully merging this pull request may close these issues.

Firefox sidebar contains links which are redundant
3 participants