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

Preserve z-index of windows when restoring from the session #10473

Closed
NejcZdovc opened this issue Aug 14, 2017 · 0 comments
Closed

Preserve z-index of windows when restoring from the session #10473

NejcZdovc opened this issue Aug 14, 2017 · 0 comments
Assignees
@NejcZdovc
Labels
QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Milestone
0.20.x (Release C...

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Aug 14, 2017 •
edited by bsclifton
Loading

Test plan

  1. Launch brave and open a few tabs
  • facebook.com
  • twitter.com
  • brave.com
  1. Resize the main window to a very specific size (ex: 800x600)
  2. Tear off the twitter and brave tabs, each into their own windows
  3. Resize those windows and stack them in an order that you can remember
  • put facebook on bottom
  • put twitter over the facebook window
  • put brave over the twitter window
  1. Quit Brave and relaunch
  2. Z-order should be preserved exactly (brave on top, then twitter, then facebook)

Original issue description

Describe the issue you encountered:
When you close multiple windows with Menu->Quit Brave and you start browser again, windows don't have the same z-index as they had before.
@NejcZdovc NejcZdovc added this to the 0.20.x (Developer Channel) milestone Aug 14, 2017
@NejcZdovc NejcZdovc self-assigned this Aug 14, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 18, 2017
@NejcZdovc
Adds z-index for windows when restoring from the session
d238b26 
Resolves brave#10473

Auditors:

Test Plan:
@NejcZdovc NejcZdovc mentioned this issue Aug 18, 2017
Merged
8 tasks
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 29, 2017
@NejcZdovc
Adds z-index for windows when restoring from the session
49960ea 
Resolves brave#10473

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 29, 2017
@NejcZdovc
Adds z-index for windows when restoring from the session
e808e2e 
Resolves brave#10473

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 29, 2017
@NejcZdovc
Adds z-index for windows when restoring from the session
db15a3f 
Resolves brave#10473

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 29, 2017
@NejcZdovc
Adds z-index for windows when restoring from the session
6ef231b 
Resolves brave#10473

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Aug 29, 2017
@NejcZdovc
Adds z-index for windows when restoring from the session
2eeb806 
Resolves brave#10473

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Sep 18, 2017
@NejcZdovc
Adds z-index for windows when restoring from the session
827d034 
Resolves brave#10473

Auditors:

Test Plan:
bsclifton pushed a commit to NejcZdovc/browser-laptop that referenced this issue Sep 18, 2017
@NejcZdovc @bsclifton
Adds z-index for windows when restoring from the session
d645dd7 
Resolves brave#10473

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Sep 19, 2017
@NejcZdovc
Adds z-index for windows when restoring from the session
2f0dcfb 
Resolves brave#10473

Auditors:

Test Plan:
cezaraugusto added a commit that referenced this issue Sep 22, 2017
@cezaraugusto
Merge pull request #10580 from NejcZdovc/feature/#10473-zindex
843d9b3 
Windows are restored with the correct z-index
cezaraugusto added a commit that referenced this issue Sep 22, 2017
@cezaraugusto
Merge pull request #10580 from NejcZdovc/feature/#10473-zindex
8e370de 
Windows are restored with the correct z-index
syuan100 pushed a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
@NejcZdovc @syuan100
Adds z-index for windows when restoring from the session
201f1a3 
Resolves brave#10473

Auditors:

Test Plan:
This was referenced Dec 20, 2017
Closed
Closed
@kjozwiak kjozwiak mentioned this issue Jan 31, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@NejcZdovc NejcZdovc

Labels
QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Projects
None yet
Milestone
0.20.x (Release Channel)
Development

No branches or pull requests
5 participants
@luixxiul @bsclifton @NejcZdovc @srirambv @LaurenWags