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

[Bug]: Calling removeBrowserView() in any way disables dragging using other BrowserViews on macOS #31068

Closed
3 tasks done
devinbinnie opened this issue Sep 22, 2021 · 2 comments · Fixed by #31085
Closed
3 tasks done

Comments

@devinbinnie
Copy link

Preflight Checklist

Electron Version

14.0.1

What operating system are you using?

macOS

Operating System Version

macOS Big Sur 11.6

What arch are you using?

x64

Last Known Working Electron version

Unsure

Expected Behavior

BrowserViews containing draggable areas should continue to function normally regardless of other BrowserViews being removed using removeBrowserView

Actual Behavior

BrowserViews containing draggable areas cease to function on macOS once removeBrowserView is called.

Testcase Gist URL

No response

Additional Information

Calling setBounds on any view seems to restore the functionality.

Reproduction using electron-quick-start here: https://github.com/devinbinnie/electron-quick-start/tree/remove_browser_view_issue

@codebytere
Copy link
Member

codebytere commented Sep 23, 2021

@devinbinnie in the future where possible could you please provide reproducible samples as Electron Fiddles? it's much easier for us - that's what the "Testcase Gist URL" is for :)

Gist Version: https://gist.github.com/0a53108546ef22e4d03e1b6dea55a79a

@devinbinnie
Copy link
Author

@devinbinnie in the future where possible could you please provide reproducible samples as Electron Fiddles? it's much easier for us - that's what the "Testcase Gist URL" is for :)

Gist Version: https://gist.github.com/0a53108546ef22e4d03e1b6dea55a79a

Absolutely, apologies for the inconvenience. Thanks so much for the quick fix :)

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

Successfully merging a pull request may close this issue.

2 participants