-
Notifications
You must be signed in to change notification settings - Fork 15.2k
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
providing option to undock devtools from browser-window #1307
Conversation
@_openDevTools() | ||
BrowserWindow::openDevTools = (options={}) -> | ||
dock = if options.detach then false else true | ||
@_openDevTools(dock) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can just do @_openDevTools(options.detach)
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since its optional, wouldnt it be better to not pass undefined
? otherwise i have to put up check for it on c++ land right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah I forgot that.
I think we can do options.detach ?= false
before calling @_openDevTools
, this matches the style of how we do with default options in other places.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed, thanks!
eee40ab
to
7ccfda4
Compare
7ccfda4
to
69d958b
Compare
👍 |
providing option to undock devtools from browser-window
Looks like this should be applied to the webview tag devTools as well? |
@krainboltgreene |
Is there a particular reason it isn't wrapped in a BrowserWindow? |
👍 |
Fixes #1135