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

Reader Mode: remove UI button (uplift to 0.69.x) #3410

Closed
wants to merge 1 commit into from

Conversation

brave-builds
Copy link
Collaborator

@brave-builds brave-builds commented Sep 11, 2019

Uplift of #3397
Fixes brave/brave-browser#5951

Approved, please ensure that before merging:

  • You have checked CI and the builds, lint, and tests all pass or are not related to your PR.
  • You have tested your change on Nightly.
  • The PR milestones match the branch they are landing to.

After you merge:

  • The associated issue milestone is set to the smallest version that the changes is landed on.

@bsclifton
Copy link
Member

needs to be merged after C77

@bsclifton bsclifton closed this Sep 11, 2019
@bsclifton bsclifton reopened this Sep 11, 2019
@bsclifton bsclifton requested review from a team and removed request for bridiver, bbondy, kjozwiak, bsclifton, rebron and srirambv September 11, 2019 20:21
@bsclifton bsclifton changed the base branch from 0.70.x to 0.69.x September 12, 2019 20:33
@bsclifton bsclifton changed the title Reader Mode: remove UI button (uplift to 0.70.x) Reader Mode: remove UI button (uplift to 0.69.x) Sep 12, 2019
@bsclifton
Copy link
Member

Closing in favor of #3433 (change has been cherry-picked there)

@bsclifton bsclifton closed this Sep 16, 2019
@bsclifton bsclifton deleted the pr3397_reader-mode-remove-button_0.70.x branch September 16, 2019 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants