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

Private and Tor windows NTP UI doesn't fill completely #3322

Closed
srirambv opened this issue Feb 12, 2019 · 5 comments
Closed

Private and Tor windows NTP UI doesn't fill completely #3322

srirambv opened this issue Feb 12, 2019 · 5 comments

Comments

@srirambv
Copy link
Contributor

Description

Private and Tor windows NTP UI doesn't fill completely. Issue originally reported by @NejcZdovc

Steps to Reproduce

  1. Install beta build
  2. Open a new Tor/Private window
  3. Shows white blank space at the bottom of the page

Actual result:

Private:image
Tor:image

Expected result:

UI should cover the entire page

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 0.60.26 Chromium: 72.0.3626.96 (Official Build) beta(64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Linux/macOS

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    Yes on beta and dev

Website problems only:

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

Additional Information

cc: @rossmoody @petemill

@srirambv srirambv added bug feature/new-tab design A design change, especially one which needs input from the design team QA/Yes release-notes/include regression labels Feb 12, 2019
@btlechowski
Copy link

btlechowski commented Feb 13, 2019

Edit: Reproduced on Windows 7 x64 and Windows 10 x64 when zoomed out. Probably resolution dependent issue.

Brave 0.60.28 Chromium: 72.0.3626.96 (Official Build) beta (64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Windows

image

@kjozwiak
Copy link
Member

Noticed the same thing happening on my macOS 10.14.2 x64 last night when I was using Tor windows. Using the following Display settings:

screen shot 2019-02-13 at 5 01 50 pm

Example of the issue occurring (notice the white spaces at the bottom occurring):

screen shot 2019-02-13 at 4 59 27 pm

@NejcZdovc
Copy link
Contributor

personally I think this should be release blocking for 0.60 as it looks really bad, specially on big screen

image

@kjozwiak
Copy link
Member

@NejcZdovc agreed. Looks like it's a regression and doesn't happen on 0.59.35. I double checked 0.59.35 and couldn't reproduce. This looks pretty bad and it's an obvious regressions. We're making the experience worse than it was one release before.

CCing @rebron on thoughts re: getting this fixed in 0.60.x.

@srirambv
Copy link
Contributor Author

srirambv commented Feb 18, 2019

Verification passed on

Brave 0.60.34 Chromium: 72.0.3626.109 (Official Build) beta(64-bit)
Revision fae8db7ab9280fa6704a59980263c804f809ebd5-refs/branch-heads/3626@{#857}
OS Linux
  • Private and Tor window NTP UI fills correctly on all resolution sizes

Verification passed on

Brave 0.60.34 Chromium: 72.0.3626.109 (Official Build) beta (64-bit)
Revision fae8db7ab9280fa6704a59980263c804f809ebd5-refs/branch-heads/3626@{#857}
OS Windows 10
  • Private and Tor window NTP UI fills correctly on all resolution sizes

Verification PASSED on macOS 10.14.2 x64 using the following build:

Brave 0.60.44 Chromium: 72.0.3626.109 (Official Build) (64-bit)
Revision fae8db7ab9280fa6704a59980263c804f809ebd5-refs/branch-heads/3626@{#857}
OS Mac OS X

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

No branches or pull requests

9 participants