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: mark skipTaskbar as unsupported on Linux #33226
Conversation
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.
Looks good to me.
Release Notes Persisted
|
## Planned Breaking API Changes (19.0) | ||
|
||
*None (yet)* | ||
### Unsupported: `skipTaskbar` on Linux |
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 missed this, but all breaking changes in this document should use one of the types described at the top of the document (API Changed / Behavior Changed / Default Changed / Deprecated / Removed).
What does Unsupported
mean?
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.
Followup in #33479
Marked as unsupported in e19, now removed in e20. See #33226 for more information
Marked as unsupported in e19, now removed in e20. See #33226 for more information
* fix: remove skip-taskbar feature on Linux. Marked as unsupported in e19, now removed in e20. See #33226 for more information * docs: remove skipTaskbar note in browserWindow Co-authored-by: Charles Kerr <charles@charleskerr.com> Co-authored-by: Keeley Hammond <khammond@slack-corp.com>
Will this affect browserWindow.setSkipTaskbar? |
BREAKING CHANGE
Description of Change
Closes #33124.
skipTaskbar
has no effect on Wayland. Currently Electron uses_NET_WM_STATE_SKIP_TASKBAR
to tell the WM to hide an app from the taskbar, and this works fine on X11 but there's no equivalent mechanism in Wayland. There are some workarounds available on different systems, e.g. GNOME and KWin, but they some unacceptable tradeoffs, e.g.Window.is_skip_taskbar
requires unsafe mode.tl;dr I don't see a way to support this feature on Linux anymore.
This PR updates the documentation to reflect that it's only supported on macOS / Windows and makes an announcement in "breaking changes" (it's broken already, but better to document it here for visibility).
Checklist
npm test
passesRelease Notes
Notes: Marked
.skipTaskbar
as unsupported on Linux.