-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
[LTS] nwjs 0.16 - MacOS 10.10 - nw.global.__nw_windows not correct #5181
Comments
@rogerwang is there any other way to get all running windows (nwwindow instances)? I'm trying chrome api but it doesn't allow to access the window object (e.g. window.document) |
@rogerwang just tested on Windows and it's the same... very often some window instance will not be tracked to the __nw_windows store var. |
We'll provide nw.Window.getAll(). Before that you can use https://developer.chrome.com/apps/app_window#method-getAll We'll see the issue about __nw_windows. |
@rogerwang tested even with Instead if I use |
@rogerwang do you have any planned date for this issue solving? It could be blocking for who need to handle number of window instances. Thank you |
up |
@rogerwang tested the new version 0.17 (latest): |
Hello @rogerwang |
@AndryBray I'll try to fix it soon in 0.17. Thanks. |
does it happen only when you open windows with remote URLs? |
I only use remote urls. I'll try with local as well |
@rogerwang I've just tested the live build version https://dl.nwjs.io/live-build/09-10-2016/nw17-4c8d707-20839a2-dacfec0-65b948e/v0.17.2/ I still have the same problem (tested on Windows 7) |
Did you test with chrome.app.window.getAll()? That's what I was fixing. |
@rogerwang OK. I was still using the nw.global.__windows object instead of chrome API. Using the chrome api it seems to work. Just for info: It's a pity that we still have problems with nw.window.open callback with undefined wind value. |
@rogerwang is this bugfix applied on 0.14.x version as well? |
@rogerwang about the NULL value I found it's not related to dev console but chrome.app.window.getAll() will return NULL values if you try to reload the content (CMD+R) of window. Tested on 0.18 beta-1 as well. |
Yes, it will be applied to LTS branch too. Reopen to track this. |
@rogerwang it would be nice to have a kind of roadmap for issues just to know your estimation about the problems solving or just to know if you are working on. Any idea about this one especially? |
Will see this before 0.17.3. I was about to fix it in 0.17.2 but was On Wed, Sep 14, 2016, 9:43 PM AndryBray [email protected] wrote:
|
Google just released another Chromium stable update in the next day after the previous one. So I'm going to release 0.17.3 soon and this issue will be likely slip into 0.17.4. |
returning null is fixed in #5294 |
it does work you just have to remember that the window opens in the context of the newly focused window. if you open chrome debugger of the latest window you should see that all your windows are all open. |
My app handles the running instances of every window and it was working good with 0.14 & 0.15.
On v 0.16 I see there is some problem with
nw.global.__nw_windows
It doesn't store always the right number of opened windows.
E.g. if I open 5 windows I see just 2 NWWindow object inside
nw.global.__nw_windows
The text was updated successfully, but these errors were encountered: