SafeEyes icon indicating copy to clipboard operation
SafeEyes copied to clipboard

The settings window should come to the front

Open AdamPS opened this issue 1 year ago • 4 comments

Describe the bug The settings window should come to the front and be visible.

See https://github.com/slgobinath/SafeEyes/issues/643#issuecomment-2348972107

To Reproduce

  1. Click on settings in the tray, leave the window open, focus another window on top of it.
  2. Click on settings again. Nothing happens.

Expected behavior The settings window comes to the front and is visible.

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 avatar Sep 18 '24 08:09 AdamPS

Please check if the error is present in the latest development version https://launchpad.net/~safeeyes-team/+archive/ubuntu/safeeyes-daily/+files/safeeyes_2.2.3-2-dev~202503041634~ubuntu24.04.1_all.deb

archisman-panigrahi avatar Mar 06 '25 16:03 archisman-panigrahi

I assume it is - see my previous comment on #643 on what would be needed for this:

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.

deltragon avatar Mar 06 '25 16:03 deltragon

I tried to run the latest dev from source as that's much easier for me, and I assume it produces the same result. I hit #693.

AdamPS avatar Mar 10 '25 14:03 AdamPS

I confirm the error is present on the current master

AdamPS avatar Mar 15 '25 15:03 AdamPS

Great thanks

AdamPS avatar Oct 21 '25 11:10 AdamPS