Bug 512034 - mnemonic mishap
Summary: mnemonic mishap
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: xchat
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Kevin Kofler
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 515500 516141 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-07-16 00:46 UTC by Matthias Clasen
Modified: 2009-08-08 19:28 UTC (History)
6 users (show)

Fixed In Version: 2.8.6-11.fc11
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-08-08 19:28:34 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
patch (580 bytes, patch)
2009-07-16 00:46 UTC, Matthias Clasen
no flags Details | Diff
Patch using gtk_button_set_use_underline (574 bytes, patch)
2009-07-18 11:42 UTC, Edward Sheldrake
no flags Details | Diff

Description Matthias Clasen 2009-07-16 00:46:42 UTC
Created attachment 353926 [details]
patch

Xchats network dialog has a C_onnect button. This happens new because we fixed a bug in GTK+ where it whould apply mnemonics even though it was not supposed to.

The attached patch should fix this (untested)

Comment 1 Kevin Kofler 2009-07-16 19:07:36 UTC
OK, I'm applying this trivial fix in Fedora, but this needs to be fixed upstream too. CCing upstream.

Comment 2 Kevin Kofler 2009-07-16 22:23:22 UTC
Should be fixed in 2.8.6-9.fc12 which is now building.

Comment 3 Edward Sheldrake 2009-07-18 11:42:30 UTC
Created attachment 354251 [details]
Patch using gtk_button_set_use_underline

I was seeing this issue on Fedora 11 with the latest testing gtk2 (2.16.4, 2.16.5), rebuilding 2.8.6-9.fc12 on Fedora 11 did not fix it, it just gives:

(xchat:27688): Gtk-CRITICAL **: gtk_label_set_use_underline: assertion `GTK_IS_LABEL (label)' failed

A working patch uses gtk_button_set_use_underline instead.

Comment 4 Kevin Kofler 2009-07-18 20:16:26 UTC
Thanks for the fixed patch, should be fixed in 2.8.6-10.fc12 which is now building.

(<rant>This bug (the incorrect patch) would not have happened if an actual object-oriented language like C++ had been used instead of abusing C for OOP. Neither Qt nor GTK-- allow this class of bugs to happen.</rant>)

Comment 5 Matthias Clasen 2009-07-18 20:19:25 UTC
Sorry about that. The corrected patch is obviously correct.

Comment 6 Kevin Kofler 2009-07-18 21:12:43 UTC
Build completed, should be fixed for good now, please reopen if the problem persists.

Comment 7 Kevin Kofler 2009-08-04 15:06:43 UTC
*** Bug 515500 has been marked as a duplicate of this bug. ***

Comment 8 Kevin Kofler 2009-08-07 12:37:02 UTC
Reopening, needs fixing in F11 too.

Comment 9 Kevin Kofler 2009-08-07 12:38:47 UTC
*** Bug 516141 has been marked as a duplicate of this bug. ***

Comment 10 Fedora Update System 2009-08-07 13:05:23 UTC
xchat-2.8.6-11.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/xchat-2.8.6-11.fc11

Comment 11 Fedora Update System 2009-08-08 19:28:23 UTC
xchat-2.8.6-11.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.