Ticket #53 (closed defect: fixed)

Opened 5 years ago

Last modified 4 years ago

"Hide on lose focus" function doesn't work.

Reported by: idyllic Owned by: GabrielFalcao
Priority: critical Milestone:
Component: guake Version:
Keywords: hide on lose focuse Cc:

Description

Hi, I built and installed from scratch Guake 0.3. Everything works smoothly. Only "Hide on lose focus" function doesn't work - as I tried Alt-Tab and switch desktop to change focus from Guake. There is no error message from the terminal =/

I'm on Ubuntu Hardy 8.04.1 with all the latest updates.

Thanks for a great program.

Change History

Changed 5 years ago by bdusauso

I confirm this behaviour. I have nearly exactly the same system (Hardy 8.04.1 with latest updates) but I've installed the debian package of Guake.

Changed 5 years ago by GabrielFalcao

  • owner changed from somebody to GabrielFalcao
  • priority changed from major to critical
  • status changed from new to assigned

I am going to review this functionallity

Changed 5 years ago by gwrynn

Could it be that the fix is as simple as this patch? The window should be visible to hide() it...

(sorry for pasting stuff here, but attaching a file in trac failed for me...)

diff --git a/src/guake.py b/src/guake.py
index 5f5f8d9..f48f3bb 100644
--- a/src/guake.py
+++ b/src/guake.py
@@ -704,7 +704,7 @@ class Guake(SimpleGladeApp):
     def on_window_lostfocus(self,window, event):
         getb = lambda x:self.client.get_bool(x)
         value = getb(GCONF_PATH+'general/hide_on_lost_focus')
-        if value and not self.visible:
+        if value and self.visible:
             self.hide()
 
     def check_widgets_visibility(self):
-- 

Changed 4 years ago by lincoln

  • status changed from assigned to closed
  • resolution set to fixed

Thank you for your contribution, this was fixed some commits ago =)

Note: See TracTickets for help on using tickets.