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.

Sign up for GitHub

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

Jump to bottom

Brave Today Disable Content menu is cut on some sections #13112

Closed
btlechowski opened this issue Dec 9, 2020 · 1 comment
Closed

Brave Today Disable Content menu is cut on some sections #13112

btlechowski opened this issue Dec 9, 2020 · 1 comment
Labels
feature/brave-news formerly brave-today feature/new-tab OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Test-Plan-Specified QA/Yes

Comments

@btlechowski
Copy link

Found when testing #12608

Steps to Reproduce

  1. Scroll down Brave Today Content till publisher name is at the top of section (check screen shot below )
  2. Click publisher name to show Disable Content menu

Actual result:

The menu is cut
image

Expected result:

The menu is not cut

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 1.18.70 Chromium: 87.0.4280.101 (Official Build) (64-bit)
Revision 9407c80213cda69c2b7abcb4fa8e3f74488f4956-refs/branch-heads/4280@{#1807}
OS Windows 7 Service Pack 1 (Build 7601.24544)
Brave 1.20.17 Chromium: 87.0.4280.88 (Official Build) nightly (64-bit)
Revision 89e2380a3e36c3464b5dd1302349b1382549290d-refs/branch-heads/4280@{#1761}
OS Windows 7 Service Pack 1 (Build 7601.24544)

cc @brave/legacy_qa @rebron @petemill
@btlechowski btlechowski changed the title Brave Today Disable Content menu is cut on some content Brave Today Disable Content menu is cut on some sections Dec 9, 2020
@btlechowski btlechowski mentioned this issue Dec 9, 2020
Closed
@btlechowski btlechowski added the feature/brave-news formerly brave-today label Dec 9, 2020
@mattmcalister mattmcalister added the priority/P3 The next thing for us to work on. It'll ride the trains. label Mar 8, 2022
@mattmcalister
Copy link

Closing as duplicate of #14196

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature/brave-news formerly brave-today feature/new-tab OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Test-Plan-Specified QA/Yes
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mattmcalister @btlechowski