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
docs: fix electron.d.ts typings #13841
Conversation
1e9a037
to
ae1f6ad
Compare
ae1f6ad
to
5104f9f
Compare
@zeke can you please review? |
@@ -194,7 +194,7 @@ Sets the `image` associated with this tray icon. | |||
|
|||
#### `tray.setPressedImage(image)` _macOS_ | |||
|
|||
* `image` [NativeImage](native-image.md) | |||
* `image` ([NativeImage](native-image.md) | String) |
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.
Instead of documenting everywhere that NativeImage
can also be a string
I think it makes more sense to remove this duplication and update electron-typescript-definitions
to alias NativeImage
to NativeImage | String
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.
Note this won't block this PR, just a thing we should probably do in the future
We have automatically backported this PR to "1-8-x", please check out #13855 |
We have automatically backported this PR to "2-0-x", please check out #13856 |
We have automatically backported this PR to "3-0-x", please check out #13857 |
Fixes #13577
win.setOverlayIcon(overlay, description)
- overlay can benull
tray.setPressedImage(image)
- image can bestring
Checklist
(https://github.com/electron/electron/blob/master/docs/development/testing.md)