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

unnecessary scroll bar on sections of Customize Dashboard popup #11374

Closed
LaurenWags opened this issue Aug 21, 2020 · 7 comments · Fixed by brave/brave-core#6634
Closed

unnecessary scroll bar on sections of Customize Dashboard popup #11374

LaurenWags opened this issue Aug 21, 2020 · 7 comments · Fixed by brave/brave-core#6634

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Aug 21, 2020

Description

Found while testing #10318

There's an unnecessary scroll bar on the Customize Dashboard popup on NTP. It makes sense when "More Cards" is selected, but not for the others (Background Image, Brave Stats, etc).

Note, I'm using Mojave (10.14.6)

Steps to Reproduce

  1. Install 1.13.x
  2. Open NTP
  3. Click on Customize

Actual result:

See scroll bar. Same for Brave Stats, Top Sites, and Clock.

Screen Shot 2020-08-21 at 3 34 52 PM

Expected result:

Scroll bar only when needed, ex. on More Cards
Screen Shot 2020-08-21 at 3 43 53 PM

Reproduces how often:

easily

Brave version (brave://version info)

Brave 1.13.79 Chromium: 85.0.4183.69 (Official Build) dev (64-bit)
Revision 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS macOS Version 10.14.6 (Build 18G3020)

Version/Channel Information:

  • Can you reproduce this issue with the current release? n/a
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

cc @ryanml @karenkliu @rebron @brave/legacy_qa

@rebron
Copy link
Collaborator

rebron commented Aug 21, 2020

Not reproducible in the following build:

Brave 1.15.2 Chromium: 85.0.4183.69 (Official Build) nightly (64-bit)
Revision 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS macOS Version 10.16 (Build 20A5354i)

and

Brave 1.13.78 Chromium: 85.0.4183.69 (Official Build) beta (64-bit)
Revision 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS macOS Version 10.16 (Build 20A5354i)

screen_shot_2020-08-21_at_12 46 31_pm

This looks to be Mojave specific and not effecting 10.15 Catalina or 10.16 Big Sur.

@rebron
Copy link
Collaborator

rebron commented Aug 21, 2020

Windows:

Revision | 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS | Windows 10 OS Version 2004 (Build 19041.450)

windows-scroll

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Aug 25, 2020

Reproduced Windows 10 x64 - 1.13.79
image

@btlechowski
Copy link

Reproduced on Linux

Brave 1.15.14 Chromium: 85.0.4183.83 (Official Build) nightly (64-bit)
Revision 94abc2237ae0c9a4cb5f035431c8adfb94324633-refs/branch-heads/4183@{#1658}
OS Linux

image

@srirambv
Copy link
Contributor

+1 from @phoogm via #11480 for additional scroll bars

bsclifton added a commit to brave/brave-core that referenced this issue Sep 13, 2020
@bsclifton bsclifton self-assigned this Sep 13, 2020
@bsclifton bsclifton added QA/Test-Plan-Specified priority/P3 The next thing for us to work on. It'll ride the trains. labels Sep 14, 2020
@srirambv
Copy link
Contributor

@LaurenWags
Copy link
Member Author

LaurenWags commented Oct 2, 2020

Verified passed with

Brave | 1.16.50 Chromium: 86.0.4240.55 (Official Build) dev (x86_64)
-- | --
Revision | a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS | macOS Version 10.14.6 (Build 18G3020)

Verified STR from description.

Confirmed scroll bar is not shown Background Image, Brave Stats, Top Sites, and Clock sections of Customize Dashboard, example:
Screen Shot 2020-10-02 at 3 36 08 PM

Confirmed scroll bar is shown on More Cards of Customize Dashboard:
Screen Shot 2020-10-02 at 3 36 21 PM


Verification passed on

Brave 1.16.50 Chromium: 86.0.4240.55 (Official Build) dev (x86_64)
Revision a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS Ubuntu 18.04 LTS

Verified STR from description.

Confirmed scroll bar is not shown Background Image, Brave Stats, Top Sites, and Clock sections of Customize Dashboard, example:
image

Confirmed scroll bar is shown on More Cards of Customize Dashboard:
image


Verification passed on

Brave | 1.16.58 Chromium: 86.0.4240.75 (Official Build) dev (64-bit)
-- | --
Revision | c69c33933bfc72a159aceb4aeca939eb0087416c-refs/branch-heads/4240@{#1149}
OS | Windows 10 OS Version 1903 (Build 18362.1082)

Verified STR from description.

Confirmed scroll bar is not shown Background Image, Brave Stats, Top Sites, and Clock sections of Customize Dashboard, example:
image

Confirmed scroll bar is shown on More Cards of Customize Dashboard:
image

@rebron rebron changed the title [Desktop] unnecessary scroll bar on sections of Customize Dashboard popup unnecessary scroll bar on sections of Customize Dashboard popup Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment