Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Added mobile version for new tab #7415

Merged
merged 1 commit into from
Mar 3, 2017

Conversation

NejcZdovc
Copy link
Contributor

@NejcZdovc NejcZdovc commented Feb 28, 2017

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

Resolves #7411

Auditors

@cezaraugusto @bsclifton @bradleyrichter

Test Plan

  • open new tab
  • resize window to 560x680
  • there shouldn't be x scroll

End result

image

Resolves brave#7411

Auditors: @cezaraugusto @bsclifton

Test Plan:
- open new tab
- resize window to 560x680
- there shouldn't be x scroll
Copy link
Contributor

@cezaraugusto cezaraugusto left a comment

Choose a reason for hiding this comment

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

super nice LGTM

@bradleyrichter
Copy link
Contributor

looks great. thanks!

Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

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

Great job! 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants