Skip to content

Flatpak aware libappindicator#371

Open
bbhtt wants to merge 1 commit intomasterfrom
bbhtt/libappindicator-flatpak-icon
Open

Flatpak aware libappindicator#371
bbhtt wants to merge 1 commit intomasterfrom
bbhtt/libappindicator-flatpak-icon

Conversation

@bbhtt
Copy link

@bbhtt bbhtt commented May 17, 2025

No description provided.

@flathubbot
Copy link
Contributor

🚧 Test build enqueued.

@flathubbot
Copy link
Contributor

🚧 Started test build.

@flathubbot
Copy link
Contributor

🚧 Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/185841/com.dropbox.Client.flatpakref

@bbhtt bbhtt force-pushed the bbhtt/libappindicator-flatpak-icon branch from dbe4e52 to 44209de Compare May 17, 2025 04:36
@flathubbot
Copy link
Contributor

🚧 Test build enqueued.

@flathubbot
Copy link
Contributor

🚧 Started test build.

@bbhtt bbhtt force-pushed the bbhtt/libappindicator-flatpak-icon branch from 44209de to 481dd93 Compare May 17, 2025 04:39
@flathubbot
Copy link
Contributor

🚧 Test build enqueued.

@flathubbot
Copy link
Contributor

🚧 Test build was cancelled.

@flathubbot
Copy link
Contributor

🚧 Started test build.

@flathubbot
Copy link
Contributor

🚧 Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/185844/com.dropbox.Client.flatpakref

@bbhtt bbhtt force-pushed the bbhtt/libappindicator-flatpak-icon branch from 481dd93 to 738ec52 Compare May 17, 2025 05:21
@flathubbot
Copy link
Contributor

🚧 Test build enqueued.

@flathubbot
Copy link
Contributor

🚧 Started test build.

@flathubbot
Copy link
Contributor

🚧 Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/185850/com.dropbox.Client.flatpakref

@bbhtt bbhtt force-pushed the bbhtt/libappindicator-flatpak-icon branch from 738ec52 to 142a481 Compare May 17, 2025 05:52
@flathubbot
Copy link
Contributor

🚧 Test build enqueued.

@flathubbot
Copy link
Contributor

🚧 Started test build.

@flathubbot
Copy link
Contributor

🚧 Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/185851/com.dropbox.Client.flatpakref

@wjt
Copy link
Member

wjt commented May 29, 2025

At least for me this didn't work – the icon was still missing in most cases.

I would take a different approach of rewriting the icon theme path from an /app/... path to the real path of the installation, via .flatpak-info.

@bbhtt
Copy link
Author

bbhtt commented May 29, 2025

It's doing that exactly and seems to work for me.

@bbhtt
Copy link
Author

bbhtt commented May 29, 2025

Are you sure you are on my branch of shared-modules?

@wjt wjt force-pushed the bbhtt/libappindicator-flatpak-icon branch from 142a481 to f667b17 Compare May 29, 2025 11:37
@flathubbot
Copy link
Contributor

🚧 Test build enqueued.

@wjt
Copy link
Member

wjt commented May 29, 2025

I must have misread... I'll check.

@flathubbot
Copy link
Contributor

🚧 Started test build.

@flathubbot
Copy link
Contributor

🚧 Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/188541/com.dropbox.Client.flatpakref

@wjt
Copy link
Member

wjt commented May 29, 2025

For me, the new build above doesn't work. When the menu first loads it gets the (blue) Dropbox icon but as soon as I try to interact with the menu it returns to the ••• icon.

Looking with Bustle, when the icon changes, Dropbox emits: org.kde.StatusNotifierItem.NewIcon (which has no arguments). Then the shell plugin queries the ("org.kde.StatusNotifierItem", "IconName") and ("org.kde.StatusNotifierItem", "IconAccessibleDesc") properties, but nothing to do with the theme path. I wonder if it's a bug in the shell extension..

@wjt
Copy link
Member

wjt commented May 29, 2025

So GetAll on /org/ayatana/NotificationItem/dropbox_client_7 includes:

"IconThemePath": <"/sysroot/home/wjt/.local/share/flatpak/app/com.dropbox.Client/x86_64/test/0830fc6aa82491ba5f10e1fc2d7dd684a5d3063124a191d0d8645a8f717fe180/files/share/icons">

but GetAll on /org/ayatana/NotificationItem/dropbox_client_7/Menu with interface ("com.canonical.dbusmenu",) gets:

({"IconThemePath": <@as []>, "Status": <"normal">, "TextDirection": <"ltr">, "Version": <uint32 3>},)

So I wonder if this is resetting the extension's idea of the icon theme path.

@bbhtt
Copy link
Author

bbhtt commented May 29, 2025

I see, i will take a look. I couldn't get the menu to show up for me.

@bbhtt
Copy link
Author

bbhtt commented May 30, 2025

Yea I'm getting no menu, what does it need to show up?

Screencast.mp4

@wjt
Copy link
Member

wjt commented May 30, 2025

Weird. I have GNOME Shell 43 with whatever version of the indicator extension is in Debian Stable, and it Simply Works.

@flathubbot
Copy link
Contributor

Test build committed. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/185844/com.dropbox.Client.flatpakref

@flathubbot
Copy link
Contributor

Test build committed. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/185841/com.dropbox.Client.flatpakref

@flathubbot
Copy link
Contributor

Test build committed. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/185851/com.dropbox.Client.flatpakref

@flathubbot
Copy link
Contributor

Test build committed. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/185844/com.dropbox.Client.flatpakref

@RichardFevrier
Copy link

The latest artifacts seems unavailable, anyway to re-run the build? I would like to test if on my machine. Thank you

@wjt
Copy link
Member

wjt commented Jul 15, 2025 via email

@flathubbot
Copy link
Contributor

🚧 Test build enqueued.

@flathubbot
Copy link
Contributor

🚧 Started test build.

@flathubbot
Copy link
Contributor

Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/198348/com.dropbox.Client.flatpakref

@RichardFevrier
Copy link

RichardFevrier commented Jul 15, 2025

Here is my testing result:

The current stable version released on flathub doesnt show any icon.
The one build with this PR shows the dropbox icon only if the device is not linked to your account (logout status). As soon as you link your device the icon disappear.
Both scenarios, I am able to interact with the menu.

Don't hesitate if you need more infos.

Environment:
OS: NixOS
Desktop Environment: Niri
Status bar: Waybar

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants