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

Maximizing window puts the bottom 1px of window behind the taskbar #7215

Closed
srirambv opened this issue Feb 13, 2017 · 5 comments · Fixed by brave/muon#191
Closed

Maximizing window puts the bottom 1px of window behind the taskbar #7215

srirambv opened this issue Feb 13, 2017 · 5 comments · Fixed by brave/muon#191

Comments

@srirambv
Copy link
Collaborator

srirambv commented Feb 13, 2017

Test plan

  1. Launch Brave on Windows and maximize the window
  2. Open a page which has a lot of white on it (something easy to see behind the taskbar) and make sure your taskbar is partially see through (aero style). Example below uses the new tab page
  3. Window contents should not be behind the taskbar
  4. Try moving the taskbar to the top and ensure there are no issues there either

Original issue description

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Maximizing window puts the bottom 1px of window behind the taskbar

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 x64

  • Brave Version (revision SHA):
    Brave 0.13.3
    rev 40d9979
    Existing issue verified till 0.12.15

  • Steps to reproduce:

    1. Ensure task bar is placed in its default position(bottom)
    2. Install Brave
    3. Click on maximize button, 1px size of window is hidden behind the taskbar
  • Actual result:
    1px size of window is hidden behind the taskbar

  • Expected result:
    Complete window should be above the taskbar when maximized

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    on 0.12.15
    image
    0.13.3 RC2
    image

  • Any related issues:
    cc: @bsclifton

@srirambv srirambv added this to the 0.13.5 milestone Feb 13, 2017
@bsclifton
Copy link
Member

@srirambv interesting that you opened this up- I was just reviewing a similar report in Electron:
electron/electron#4045

Basically, it's a problem upstream in Chromium. It seems the ticket is close though- I'll see if I can give it a bump

@bsclifton bsclifton removed this from the 0.13.5 milestone Feb 13, 2017
@bsclifton
Copy link
Member

For reference, here's the bug being tracked in Chromium:
https://bugs.chromium.org/p/chromium/issues/detail?id=576192

@lukemulks
Copy link
Collaborator

We've received a similar report from a contributor in the Brave Community:
https://community.brave.com/t/youtube-skip-ad-button-disappeared-brave-window-overlaps-other-screen/2329/9

Brave: 0.15.0 
rev: 71fb73001617949528cc6473e288a49e76fcd4d5 
Muon: 2.58.7 
libchromiumcontent: 58.0.3029.81 
V8: 5.8.283.32 
Node.js: 7.9.0 
Update Channel: dev 
os.platform: win32 
os.release: 10.0.15063 
os.arch: x64

Contributor Feedback:

Steps to reproduce:
unclear - not sure, doesn't work every time
1. minimze brave window
2. open it up again

windows-brave-taskbar-overlap-04282017

@bsclifton bsclifton added this to the 0.15.3 milestone May 5, 2017
@bsclifton bsclifton self-assigned this May 5, 2017
bsclifton added a commit to brave/muon that referenced this issue May 5, 2017
- manually applied: electron/electron@638eae1
- manually applied: https://github.com/electron/electron/pull/9167/files

Passes the correct subsystem variable to linker. Before, older Windows version
was being used which affected GetSystemMetrics (wrong values returned)

Should fix brave/browser-laptop#7641
Should fix brave/browser-laptop#7215
Might fix brave/browser-laptop#5159

Auditors: @bbondy, @bridiver
bsclifton added a commit to brave/muon that referenced this issue May 5, 2017
- manually applied: electron/electron@638eae1
- manually applied: https://github.com/electron/electron/pull/9167/files

Passes the correct subsystem variable to linker. Before, older Windows version
was being used which affected GetSystemMetrics (wrong values returned)

Should fix brave/browser-laptop#7641
Should fix brave/browser-laptop#7215
Might fix brave/browser-laptop#5159

Auditors: @bbondy, @bridiver
@srirambv
Copy link
Collaborator Author

@bsclifton This works as expected but as a result when the browser is maximised and relaunched, there is a 1px gap between the browser window and the taskbar and the browser is in a maximised restore position and will have to maximised again to fit in screen correctly

@bsclifton
Copy link
Member

@srirambv thanks for the heads up (and for creating the issue 😄 )

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