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

pinned tabs don't load correctly on new windows #148

Closed
bridiver opened this issue Jan 13, 2016 · 10 comments
Closed

pinned tabs don't load correctly on new windows #148

bridiver opened this issue Jan 13, 2016 · 10 comments

Comments

@bridiver
Copy link
Collaborator

Ideally the pinned tab should literally move from window to window and maintain state, but that will require guest view from #137 but right now if you open a new window the page doesn't load correctly
Steps to reproduce:

  1. Open cnn.com
  2. Pin the tab
  3. Open a new window
    The pinned tab will be missing css, etc...
@bbondy
Copy link
Member

bbondy commented Jan 13, 2016

In the current design each window loads a separate copy of the page. I agree this new design would be more ideal though.

Pls confirm the page does load, just a different copy right now for you. If not let me know what your pinned site is.

@bridiver
Copy link
Collaborator Author

it doesn't load in the other window for me, at least not completely. Appears to be missing images, css, etc.. also seems to trigger this when closing

Uncaught Exception:
Error: Object has been destroyed
    at Error (native)
    at Function.<anonymous> (/Users/bjohnson/Documents/brave/browser-electron/app/filtering.js:32:12)

@bbondy
Copy link
Member

bbondy commented Jan 13, 2016

Loads a second copy of the site for me, not sure what that error is but could be related. Could you give an exact url of the site you are pinning? I'm pinning apple.com.

@bridiver
Copy link
Collaborator Author

apple.com also works for me, but facebook.com fails to load completely

@bbondy
Copy link
Member

bbondy commented Jan 14, 2016

who needs facebook ;)

@bbondy
Copy link
Member

bbondy commented Jan 14, 2016

facebook also working for me :(

@luixxiul
Copy link
Contributor

Was this fixed?

@bridiver
Copy link
Collaborator Author

I still see an issue with Facebook
After pinning tab
fb1
Pinned tab in new window
fb2

@luixxiul
Copy link
Contributor

Um, on Windows 10 it cannot be reproduced. I pinned facebook.com and opened a new window, but pinned facebook was displayed correctly. OS X only bug maybe?

@bridiver
Copy link
Collaborator Author

fixed by 6e307bc

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

No branches or pull requests

3 participants