Skip to content
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

window.bringToTop() doesn't work #36

Closed
ghost opened this issue Jan 23, 2020 · 5 comments
Closed

window.bringToTop() doesn't work #36

ghost opened this issue Jan 23, 2020 · 5 comments
Labels
bug Something isn't working

Comments

@ghost
Copy link

ghost commented Jan 23, 2020

Bug description
window.bringToTop flashes the window in the task bar as if it was focused, but doesn't bring it forward

To Reproduce
image

Expected behavior
Bring to top should bring a window to top

Details:

  • Operating System: Windows 10
  • node-window-manager version: 2.0.1
  • Last known working node-window-manager version: Never used this before today

Additional context
Working in electron, if that's relevant

@ghost ghost added the bug Something isn't working label Jan 23, 2020
@ghost
Copy link
Author

ghost commented Jan 23, 2020

Actually doing just about anything in electron breaks this. Minimizing a window, then using restore, for example, freezes the application

@sentialx
Copy link
Owner

I also observed that bringing Electron BrowserWindows to top doesn't work.

@stefnotch
Copy link
Contributor

This answer on StackOverflow does appear to work.

@arejaycola
Copy link
Contributor

@stefnotch Do you have an example of how that would look with this library?

@stefnotch
Copy link
Contributor

You'd have to modify the native code of this library. Sadly I'm not well versed enough in cpp to do this and then submit a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants