Bug 510547 - geo:// handlers should have StartupNotify=true
Summary: geo:// handlers should have StartupNotify=true
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kguiaddons
Classification: Frameworks and Libraries
Component: general (other bugs)
Version First Reported In: 6.18.0
Platform: NixOS Linux
: NOR normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-10-13 00:55 UTC by Ilya Fedin
Modified: 2025-10-21 22:03 UTC (History)
1 user (show)

See Also:
Latest Commit: https://invent.kde.org/frameworks/kguiaddons/-/commit/44685fb1b9ab09aa748d56c7f6cdfbd2775b0ef7
Version Fixed/Implemented In: 6.20
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Ilya Fedin 2025-10-13 00:55:54 UTC 
SUMMARY
GLib doesn't set XDG_ACTIVATION_TOKEN unless StartupNotify=true is set: https://gitlab.gnome.org/GNOME/glib/-/blob/d54816de375b89c23cb7937f2153856db508d616/gio/gdesktopappinfo.c#L3010

STEPS TO REPRODUCE
Not sure how this could be reproduced as I don't know any application using GLib and geo://... I was attempting to implement geo:// URL support in a non-KDE application and noticed that.

OBSERVED RESULT
GLib doesn't set  XDG_ACTIVATION_TOKEN when opening geo:// URLs on KDE

EXPECTED RESULT
GLib sets  XDG_ACTIVATION_TOKEN when opening geo:// URLs on KDE

SOFTWARE/OS VERSIONS
Operating System: NixOS 25.11
KDE Plasma Version: 6.4.5
KDE Frameworks Version: 6.18.0
Qt Version: 6.9.2
Kernel Version: 6.17.0-cachyos (64-bit)
Graphics Platform: Wayland
Processors: 32 × AMD Ryzen 9 5950X 16-Core Processor
Memory: 64 ГиБ of RAM (62.7 ГиБ usable)
Graphics Processor: AMD Radeon RX 6400
Manufacturer: Micro-Star International Co., Ltd.
Product Name: MS-7C94
System Version: 1.0
Comment 1 Nate Graham 2025-10-13 19:29:59 UTC 
Wanna submit a patch?
Comment 2 Bug Janitor Service 2025-10-14 10:22:20 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kguiaddons/-/merge_requests/186
Comment 3 Bug Janitor Service 2025-10-14 10:26:06 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/education/marble/-/merge_requests/180
Comment 4 Ilya Fedin 2025-10-20 12:36:22 UTC 
Git commit 44685fb1b9ab09aa748d56c7f6cdfbd2775b0ef7 by Ilya Fedin.
Committed on 17/10/2025 at 01:54.
Pushed by davidre into branch 'master'.

Add StartupNotify=true to geo handlers

M  +1    -0    src/geo-scheme-handler/google-maps-geo-handler.desktop
M  +1    -0    src/geo-scheme-handler/openstreetmap-geo-handler.desktop
M  +1    -0    src/geo-scheme-handler/wheelmap-geo-handler.desktop

https://invent.kde.org/frameworks/kguiaddons/-/commit/44685fb1b9ab09aa748d56c7f6cdfbd2775b0ef7
Comment 5 Albert Astals Cid 2025-10-21 19:05:19 UTC 
Git commit e2614546bf50746737233f9a136e75e481bcec6e by Albert Astals Cid, on behalf of Ilya Fedin.
Committed on 21/10/2025 at 18:17.
Pushed by aacid into branch 'master'.

Add StartupNotify=true

This ensures GTK apps, portal and anything else using GLib provides XDG activation token

M  +1    -0    src/apps/behaim/org.kde.marble.behaim.desktop
M  +1    -0    src/apps/marble-kde/marble_geo.desktop
M  +1    -0    src/apps/marble-kde/marble_worldwind.desktop
M  +1    -0    src/apps/marble-kde/org.kde.marble.desktop
M  +1    -0    src/apps/marble-maps/org.kde.marble.maps.desktop
M  +1    -0    src/apps/marble-qt/org.kde.marble-qt.desktop

https://invent.kde.org/education/marble/-/commit/e2614546bf50746737233f9a136e75e481bcec6e