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

Brave is not maximized when re-launched #8600

Closed
srirambv opened this issue May 1, 2017 · 12 comments
Closed

Brave is not maximized when re-launched #8600

srirambv opened this issue May 1, 2017 · 12 comments

Comments

@srirambv
Copy link
Collaborator

srirambv commented May 1, 2017

Test plan

  1. Launch Brave on Windows
  2. Maximize the window
  3. Quit Brave and relaunch
  4. Window should open in the maximized state

Original issue description

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

  • Describe the issue you encountered:

However, I still experience this issue. I've only seen screenshots of Windows 10 and references to 'Windows' in general (and Mac OS X and Linux). So perhaps the issue is now limited to previous versions of Windows. I'm experiencing this problem on Windows 7, 64-bit. It does not happen for other programs.

User confirmed issue on clean install also

@srirambv srirambv added misc/fullscreen-option needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. OS/Windows labels May 1, 2017
@josiah-keller
Copy link
Contributor

I still have this on Windows 10. The window has the right dimensions, but it's not in the maximized state and its position is off by a few pixels.

@srirambv
Copy link
Collaborator Author

srirambv commented May 4, 2017

User(Stefan) provided additional info on the issue

When I performed clean install of Brave, initially the problem is gone.

However, when start tweaking in the settings the problem returned. (I added Lastpass and switched "Send a 'Do Not Track' header with browsing requests *")

@Degine
Copy link

Degine commented May 5, 2017

I've been playing around with clean installs and see when exactly the problem occurs. It happens when the startup setting is changed from the default setting "My windows / tabs from last time". Perhaps in addition to which tabs were opened, it also saves the exact position and layout.
image

I'm sorry if my previous remark about the Lastpass/Do not track was misleading. I must have changed the "Brave starts with" setting without paying attention.

@svenified
Copy link

svenified commented Jun 4, 2017

Maximizing the window, closing the browser, then reopening browser still results in the window opening with the correct proportions, not in the maximized state, and offset to the left and up by about 8-9 pixels. This is a fresh install on Windows 10 Home, with no settings changed. Also happens on Windows 10 Professional and Windows 7 Ultimate.
offset

@luixxiul luixxiul added the bug label Jun 5, 2017
@jonathansampson
Copy link
Collaborator

jonathansampson commented Jun 7, 2017

The maximized state is not presently being stored in the session-store-1 file. There should be a isMaximized property, according to https://github.com/brave/browser-laptop/blob/master/docs/state.md:

"perWindowState": [
    {
        "activeFrameKey": 1,
        "closedFrames": [],
        "frames": [{}],
        "hamburgerMenuWasOpen": false,
        "ui": {
            "bookmarksToolbar": {},
            "contextMenu": {
                "selectedIndex": null
            },
            "menubar": {
                "isVisible": false,
                "lastFocusedSelector": null
            },
            "tabs": {
                "tabPageIndex": 0
            }
        }
    }
],

@jonathansampson jonathansampson added this to the 0.19.x (Developer Channel) milestone Jul 24, 2017
@alexwykoff alexwykoff modified the milestones: 0.20.x (Nightly Channel), 0.19.x (Developer Channel) Jul 25, 2017
@bsclifton
Copy link
Member

When this is fixed, it would be good to write a webdriver test to confirm (can look at example code used for session store). Might be hard though, since it'll only run on Linux

@LaurenWags
Copy link
Member

@LaurenWags
Copy link
Member

@bbondy bbondy assigned NejcZdovc and unassigned bsclifton Aug 8, 2017
@alexwykoff
Copy link
Contributor

+1 from community : #8600

@eljuno
Copy link
Contributor

eljuno commented Aug 9, 2017

+1 from community
+3 from community

@NejcZdovc
Copy link
Contributor

+1 from #10455

NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 13, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 13, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 13, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 14, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 14, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 14, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 14, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 14, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 14, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 16, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 17, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 18, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 25, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 25, 2017
@bsclifton bsclifton added QA/test-plan-specified release-notes/include and removed needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. labels Aug 28, 2017
@jbrockc
Copy link

jbrockc commented Jan 27, 2018

Here is my little workaround on Debian. (I always prefer my browser maximized though.) I saved the script to my ~/bin and named it brave. This requires xdotool and wmctrl.

#!/bin/bash
open_status=$(pgrep "brave" | wc -l)
if ((open_status <= 2)); then
	/usr/bin/brave -- "$@" & sleep 2; wmctrl -r :ACTIVE: -b remove,maximized_vert,maximized_horz; sleep 1; wmctrl -r :ACTIVE: -b add,maximized_vert,maximized_horz
elif [ -n "$1" ]; then
	/usr/bin/brave -- "$@"
else
	/usr/bin/brave; xdotool key ctrl+t
fi

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