Ticket #158 (closed enhancement: fixed)

Opened 15 months ago

Last modified 11 months ago

Request Guake Confirm Quit

Reported by: phip Owned by: somebody
Priority: Fix available Milestone: 0.5.0
Component: guake Version: 0.4
Keywords: Confirm Quit Cc: makism@…

Description

I just pressed Ctrl-Shift-Q when I meant to press Ctrl-Shift-W, and Guake quit without any warning. I request a preference to confirm before quitting or confirm before closing tabs with non-shell processes active, like many terminals have.

Attachments

guake.schemas.patch (0.8 kB) - added by makism 13 months ago.
Makefile.am.patch (329 bytes) - added by makism 13 months ago.
Makefile.in.patch (294 bytes) - added by makism 13 months ago.
i`m sure if this is needed…
prefs.glade.patch (40.7 kB) - added by makism 13 months ago.
prefs.py.patch (0.9 kB) - added by makism 13 months ago.
prompt_on_quit.glade (4.7 kB) - added by makism 13 months ago.
the actual prompt dialog
guake.py.patch (3.6 kB) - added by makism 12 months ago.
updated patch. 1) hides the main window when prompting and 2) showing again on cancel. finally, 3) raises the prompt dialog on top of everything (at least in theory)

Change History

Changed 15 months ago by phip

  • priority changed from Fix available to minor
  • version set to 0.4
  • milestone 0.5.0 deleted

Changed 15 months ago by lincoln

nice suggestion!

Changed 13 months ago by pingou

  • milestone set to 0.5.0

Changed 13 months ago by makism

Changed 13 months ago by makism

Changed 13 months ago by makism

i`m sure if this is needed...

Changed 13 months ago by makism

Changed 13 months ago by makism

Changed 13 months ago by makism

the actual prompt dialog

Changed 13 months ago by makism

  • cc makism@… added

I thought it would be a great feature so i`ve made a simple patch which works well. The code for tracking how many processes are running is taken from the gnome-terminal.

The number of patches is big because the "prompt dialog" is a glade file. I could just use the gtk.Dialog class etc but i thought that i should follow the coding style and inherit from simplegladeapp. It also adds an option to the settings.

I think that maybe there is something wrong with the prefs.glade and my glade-3 binary, because after i edited it changed some things that i did not touch. But still it works fine.

I`ll be happy if i can be any of more help :-)

Changed 13 months ago by pingou

  • priority changed from minor to Fix available

Changed 12 months ago by makism

updated patch. 1) hides the main window when prompting and 2) showing again on cancel. finally, 3) raises the prompt dialog on top of everything (at least in theory)

Changed 11 months ago by lincoln

Hello there!

I'm reviewing your changes. I just loved your ideas, mainly the process count stuff.

ps.: It would be very easier to review things if you use git format-patch utility. Downloading and applying multiple patch files is not the best way =P

Changed 11 months ago by lincoln

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

Hello again!

I've just reviewed, fixed some things and pushed your patches. Thanks a lot for your help dude! To fit some of our standards I had to change some things in them. Here is a list of things I changed:

  • Coding standard fixes: removing unecessary white spaces, lower case some variable names.
  • Implemented the PromptOnQuit? class without the glade file (will help us migrate to gtk builder)
  • Removed all unrelated changes from the prefs.glade file

Thanks again, very nice job!

Note: See TracTickets for help on using tickets.