Ticket #158 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years 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 Download (0.8 KB) - added by makism 4 years ago.
Makefile.am.patch Download (329 bytes) - added by makism 4 years ago.
Makefile.in.patch Download (294 bytes) - added by makism 4 years ago.
i`m sure if this is needed…
prefs.glade.patch Download (40.7 KB) - added by makism 4 years ago.
prefs.py.patch Download (0.9 KB) - added by makism 4 years ago.
prompt_on_quit.glade Download (4.7 KB) - added by makism 4 years ago.
the actual prompt dialog
guake.py.patch Download (3.6 KB) - added by makism 4 years 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 4 years ago by phip

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

Changed 4 years ago by lincoln

nice suggestion!

Changed 4 years ago by pingou

  • milestone set to 0.5.0

Changed 4 years ago by makism

Changed 4 years ago by makism

Changed 4 years ago by makism

i`m sure if this is needed...

Changed 4 years ago by makism

Changed 4 years ago by makism

Changed 4 years ago by makism

the actual prompt dialog

Changed 4 years 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 4 years ago by pingou

  • priority changed from minor to Fix available

Changed 4 years 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 4 years 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 4 years 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.