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

Port Unsupported Systems page to bedrock #2091

Merged

Conversation

sgarrity
Copy link
Contributor

@sgarrity sgarrity commented Jun 9, 2014

Bug 988044

<div class="help-feature">
<h3>Need <span>Help?</span></h3>
<p>Our Support page has plenty of answers, including full instructions for downloading and installing Firefox and a live chat feature to guide you through any tricky spots.</p>
<a class="blocklevel" href="http://support.mozilla.org/en-US/kb/">Visit Firefox Support</a>
Copy link
Member

Choose a reason for hiding this comment

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

This redirects to https://support.mozilla.org/products/firefox (which is also localized, so we can remove the hard-coded en-US link).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. Updated to the actual destination link with HTTPS and without the locale code.

@alexgibson
Copy link
Member

r+ on the front-end code. Looks like it's just the redirect that needs updating

@sgarrity
Copy link
Contributor Author

sgarrity commented Jun 9, 2014

Updated with a redirect from unsupported-systems.html -> unsupported-systems/ (and the /b/ pass-through redirect).

@sgarrity
Copy link
Contributor Author

sgarrity commented Jun 9, 2014

@alexgibson Updated the regex to remove the unnecessary brackets.

@alexgibson
Copy link
Member

r+

alexgibson added a commit that referenced this pull request Jun 9, 2014
…stems

Port Unsupported Systems page to bedrock
@alexgibson alexgibson merged commit 5215673 into mozilla:master Jun 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants