Whitespace and style changes are making it difficult to read the actual changes.

Tiny patch to make it build on alder systems:

diff --git a/src/core/linux/SDL_dbus.h b/src/core/linux/SDL_dbus.hindex 738fc06..27dcd41 100644--- a/src/core/linux/SDL_dbus.h+++ b/src/core/linux/SDL_dbus.h@@ -38,6 +38,9 @@ #ifndef DBUS_TYPE_UNIX_FD #define DBUS_TYPE_UNIX_FD ((int)'h') #endif+#ifndef DBUS_ERROR_UNKNOWN_PROPERTY+#define DBUS_ERROR_UNKNOWN_PROPERTY "org.freedesktop.DBus.Error.UnknownProperty"+#endif  typedef struct SDL_DBusContext {

+#ifndef DBUS_ERROR_UNKNOWN_PROPERTY+#define DBUS_ERROR_UNKNOWN_PROPERTY "org.freedesktop.DBus.Error.UnknownProperty"+#endif

Implemented.

Menu export works now, I just gotta wire it up.

Will this implementation count with being able to use scroll events?

Currently we use it to set volume up/down based on scroll up/down over the tray (albeit the support for that in the current tray ecosystem seems spotty).

https://github.com/Taiko2k/Tauon/blob/e943bef475eb555b6bd5769e02ec7f11f80d2d54/src/tauon/t_modules/t_dbus.py#L269-L289

If I'm understanding theSDL docs correctly, the SDL API having its guts rewritten currently only exposes the ability to define a context menu and set an icon and tooltip.

(Which matches what I'm seeing in the diff where a quick Ctrl+F shows no mention of the D-BusSecondaryActivate orScroll signals...SecondaryActivate being middle click under KDE.)

Will this implementation count with being able to use scroll events?

Currently we use it to set volume up/down based on scroll up/down over the tray (albeit the support for that in the current tray ecosystem seems spotty).

https://github.com/Taiko2k/Tauon/blob/e943bef475eb555b6bd5769e02ec7f11f80d2d54/src/tauon/t_modules/t_dbus.py#L269-L289

If the maintainers approve, we can add this.

@slouken

Will this implementation count with being able to use scroll events?
Currently we use it to set volume up/down based on scroll up/down over the tray (albeit the support for that in the current tray ecosystem seems spotty).
https://github.com/Taiko2k/Tauon/blob/e943bef475eb555b6bd5769e02ec7f11f80d2d54/src/tauon/t_modules/t_dbus.py#L269-L289

If the maintainers approve, we can add this.

@slouken

Sure, we can add this. Go ahead and create a separate follow-on PR once this lands and include Windows support as well.

Thanks!

Two more things that I can't seem to find in the current source, is it planned to:

1. Be able to hide/show the icon without destroying it completely? AppIndicator has this option viaset_status(0) andset_status(1) -https://lazka.github.io/pgi-docs/AyatanaAppIndicator3-0.1/classes/Indicator.html#AyatanaAppIndicator3.Indicator.set_status +https://lazka.github.io/pgi-docs/AyatanaAppIndicator3-0.1/enums.html#AyatanaAppIndicator3.IndicatorStatus

2. Set icon description - this seems to be done for accessibility, unsure where/how it's actually used -https://lazka.github.io/pgi-docs/AyatanaAppIndicator3-0.1/classes/Indicator.html#AyatanaAppIndicator3.Indicator.set_icon_full

Thank you very much for working on this implementation!

Two more things that I can't seem to find in the current source, is it planned to:

1. Be able to hide/show the icon without destroying it completely? AppIndicator has this option viaset_status(0) andset_status(1) -https://lazka.github.io/pgi-docs/AyatanaAppIndicator3-0.1/classes/Indicator.html#AyatanaAppIndicator3.Indicator.set_status +https://lazka.github.io/pgi-docs/AyatanaAppIndicator3-0.1/enums.html#AyatanaAppIndicator3.IndicatorStatus
2. Set icon description - this seems to be done for accessibility, unsure where/how it's actually used -https://lazka.github.io/pgi-docs/AyatanaAppIndicator3-0.1/classes/Indicator.html#AyatanaAppIndicator3.Indicator.set_icon_full

Thank you very much for working on this implementation!

What do you think?@slouken

Two more things that I can't seem to find in the current source, is it planned to:

1. Be able to hide/show the icon without destroying it completely? AppIndicator has this option viaset_status(0) andset_status(1) -https://lazka.github.io/pgi-docs/AyatanaAppIndicator3-0.1/classes/Indicator.html#AyatanaAppIndicator3.Indicator.set_status +https://lazka.github.io/pgi-docs/AyatanaAppIndicator3-0.1/enums.html#AyatanaAppIndicator3.IndicatorStatus
2. Set icon description - this seems to be done for accessibility, unsure where/how it's actually used -https://lazka.github.io/pgi-docs/AyatanaAppIndicator3-0.1/classes/Indicator.html#AyatanaAppIndicator3.Indicator.set_icon_full

Thank you very much for working on this implementation!

What do you think?@slouken

Typically we expose features in SDL that are available on multiple platforms. If this can be implemented on Windows, then it's fine to add. If it can't, we can discuss use cases and figure out if it makes sense to add it as a platform specific feature.