kerneloops new notification dialog

I removed the bold text that said "Your system had a kernel failure" and added a window title that says "Kernel Failure".

Updated patch.

Nice work! Minor tweaks if you feel like it:

* "Don't send" should be "Don’t Send".

* Either "Send" or "Don’t Send" should be the default button. (Probably "Send", assuming that if you didn't want to submit the reports you wouldn't have kerneloops installed in the first place.)

* The message text, the disclosure triangle, and the left edge of the "Always" button should be flush left. (If the window is enlarged to show more details, the "Always" and "Never" buttons should stick to the left, while the "Don’t Send" and "Send" buttons stick to the right.)

* Fixed capitalization for "Don't Send"
* Set "Send" as the default response

- Did not change the layout, such as the alignment of widgets according to dialog edges. That would take a little bit more poking around inside the dialog's contents than I feel is worth doing right now. If someone else wants to do this, go for it.

mpt asked me to remove the window icon in the titlebar. This patch does that.

Attached debdiff with patch

Updated debdiff with the patch inline

Updated debdiff, removing test/*dbg

This bug was fixed in the package kerneloops - 0.12-0ubuntu4

---------------
kerneloops (0.12-0ubuntu4) jaunty; urgency=low

  [ Ken VanDine ]
  * debian/applied-patches/kerneloops-applet.c.patch: Convert notification to
    a dialog. Applied inline, since package has no patch system. (LP: #344377)
  * debian/rules: Remove test/*dbg, these should be cleaned by the Makefile
  * debian/applied-patches/submit.c.patch: Fix the config parser so that it
    will actually work with apport. Thanks to Matt Zimmerman for the patch!
    (LP: #344813)

  [ Martin Pitt ]
  * Moved *.patch to debian/applied-patches/, to be a little less confusing.

 -- Ken VanDine <email address hidden> Thu, 19 Mar 2009 08:48:36 -0400

Hey, I made a small mistake in the dialog. Attaching a patch next.

I was about to open a bug on the incorrect box packing just now when I found Cody's patch in the comment above. I've tested the patch and it works perfectly (I'm on xorg-edgers so I get at least one kerneloops per boot up).

Ken, will you merge this patch for jaunty?

New debdiff with Cody's patch included

This bug was fixed in the package kerneloops - 0.12-0ubuntu5

---------------
kerneloops (0.12-0ubuntu5) jaunty; urgency=low

  * debian/applied-patches/kerneloops-applet.c.patch: Updated patch to include
    a fix for box packing so expander fills and expands submitted by Cody
    Russell. (LP: #344377)

 -- Ken VanDine <email address hidden> Wed, 25 Mar 2009 15:27:17 -0400