"Show next break time in tray icon" isn't working

[AdamPS]

Describe the bug
After a recent upgrade, the setting "Show next break time in tray icon" isn't working

To Reproduce
Click on settings, select plugins, open Tray Icon settings and enable "Show next break time in tray icon". Nothing happens, I don't see the time.

Expected behavior
I see the time adjacent to the tray icon.

Desktop (please complete the following information):

• Ubuntu 24.04
• GNOME
• Version 2.2.2

Debug Log
Run the Safe Eyes using safeeyes --debug command attach the ~/safeeyes.log` file.

Screenshots
If applicable, add screenshots to help explain your problem.

[AdamPS]

I believe B was caused by #558
A and C are long-time bugs

[AdamPS]

The current code tries to put the time in a tooltip, but that's not really right. We don't want to have to hover to see the time. Anyway tooltips seem to be unsupported in AppIndicator.

The previous code used indicator.set_label()

However StatusNotifierItem seems to be missing a function for label.

[AdamPS]

Please can the main contributors for the issue that caused this regression join the discussion here?
@archisman-panigrahi @deltragon

[deltragon]

@AdamPS I have a PR up that should hopefully fix this: #646

[deltragon]

Regarding C - at least on KDE Plasma, there's a method to integrating xdg-activation into the tray icon ("ProvideXdgActivationToken"). Looking at this, it seems the GNOME extension supports it too.
IIUC, this maps to the GtkWindow.set_startup_id method, which at least for GTK4 supports xdg-activation.
I'll look into it, but it might make sense to track in a different issue.

[AdamPS]

Great thanks. I confirm that the PR works on Gnome.

[AdamPS]

Regarding C - at least on KDE Plasma, there's a method to integrating xdg-activation into the tray icon ("ProvideXdgActivationToken"). Looking at this, it seems the GNOME extension supports it too. IIUC, this maps to the GtkWindow.set_startup_id method, which at least for GTK4 supports xdg-activation. I'll look into it, but it might make sense to track in a different issue.

I raised #649

[archisman-panigrahi]

Sorry about my silence in the past several weeks. I can test the PRs tonight.
@deltragon Can you please suggest the order in which I can test/merge the existing PRs?

[deltragon]

@archisman-panigrahi #646 is probably the highest priority.

[archisman-panigrahi]

@AdamPS Can you please edit A and C out of this? I was initially super confused about whether #646 is about A and C or not.

Please create a separate issue for A.

[AdamPS]

Great thanks for the fix.

Please create a separate issue for A.

#654