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

Add history of web browser page (Fixes #6709) #6764

Merged
merged 3 commits into from
Feb 5, 2019

Conversation

alexgibson
Copy link
Member

@alexgibson alexgibson commented Jan 29, 2019

Description

  • Add new page at /technology/browser-history/
  • Also includes some small fixes for /technology/update-browser/ page:
    • Fix whitespace indentation
    • Remove hard coded locales from support links
    • Fix download button positioning in call out at small screen sizes.

Issue / Bugzilla link

#6709

Testing

Demo: https://bedrock-demo-agibson.oregon-b.moz.works/en-US/technology/browser-history/

  • Check for copy pasta errors, typos?

@alexgibson alexgibson added Do Not Merge ⚠️ P2 Second level priority - Should have Review: S Code review time: 30 mins to 1 hour labels Jan 29, 2019
@alexgibson
Copy link
Member Author

Do not merge until we get stakeholder approval.

@alexgibson
Copy link
Member Author

Rebased

@ejregithub
Copy link
Contributor

@alexgibson - we are stakeholder approved now, I've removed the DNM tag.

@alexgibson
Copy link
Member Author

I'm not quite clear if there are copy changes to be made here, so adding back the dnm label for now.

@ejregithub
Copy link
Contributor

@alexgibson - Copy is unchanged. Apologies if I presented any confusion. This is good to go.

@alexgibson
Copy link
Member Author

@ejregithub ok thanks for confirming :) This is good for review again.

Copy link
Contributor

@amychurchwell amychurchwell left a comment

Choose a reason for hiding this comment

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

One small nit, but other than that LGTM! 👍

@alexgibson
Copy link
Member Author

Thanks @amychurchwell good suggestion, updated 👍

Copy link
Contributor

@amychurchwell amychurchwell left a comment

Choose a reason for hiding this comment

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

r+ 💻

@amychurchwell amychurchwell merged commit 6fbc35f into mozilla:master Feb 5, 2019
@alexgibson alexgibson deleted the browser-history-page branch November 15, 2019 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Second level priority - Should have Review: S Code review time: 30 mins to 1 hour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants