Button press events force xchat into text selection mode that can't be broken.

Bug #528017 reported by Cody Russell
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
XChat-GNOME
Fix Released
Medium
xchat-gnome (Ubuntu)
Invalid
Undecided
Unassigned

Bug Description

Binary package hint: xchat-gnome

This is caused by the Lucid changes to gtk+, and is noticeable only if you have csd enabled.

Revision history for this message
Cody Russell (bratsche) wrote :
tags: added: patch
Revision history for this message
Cody Russell (bratsche) wrote :

Found one more case that needed a change.

Revision history for this message
Nigel Babu (nigelbabu) wrote :

Thanks for your patch. Can you open a bug report upstream and submit the patch there too?

Revision history for this message
Cody Russell (bratsche) wrote :
Revision history for this message
Nigel Babu (nigelbabu) wrote :

Thanks for submitting it upstream, could you address a few concerns? Will the proposed patch have correct behaviour when CSD is disabled too? Possible to check if CSD is enabled or disabled before returning True/False. Is CSD expected to be enabled in lucid by default. Once these concerns are addressed, I'll be happy to package and get it uploaded.

Revision history for this message
Cody Russell (bratsche) wrote :

It should not affect the operation in non-CSD mode at all.

This actually just seems to be a bug in xchat-gnome that has never been noticed because when the event propagates up to the toplevel widget there has never been one that catches it before.

It should return TRUE if it handles the button-press event.

But CSD will not be in Lucid, so keeping the bug in xchat-gnome isn't hurting anything at the moment. But it would still be nice to get it fixed in upstream. :)

Revision history for this message
Nigel Babu (nigelbabu) wrote :

Since CSD will not be in Lucid, we'll wait for upstream to integrate the patch then.

tags: added: patch-forwarded-upstream
removed: patch
Revision history for this message
Cody Russell (bratsche) wrote :

This was accepted upstream and it will be in the next release. Since we don't need this now until Lucid+1 there's no need to keep tracking it in Launchpad I think.

Changed in xchat-gnome (Ubuntu):
status: New → Invalid
Changed in xchat-gnome:
status: Unknown → Fix Released
Changed in xchat-gnome:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.