Closed Bug 955479 Opened 10 years ago Closed 10 years ago

Starting conversations on unrelated keyboard- or mouse-events

Categories

(Instantbird Graveyard :: Conversation, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: benediktp, Assigned: nhnt11)

References

Details

Attachments

(1 file)

*** Original post on bio 2042 at 2013-07-12 09:57:00 UTC ***

Double clicking the window titlebar to maximize the window on Windows starts a conversation with first contact in list.
Double clicking the window titlebar to restore a window to normal size does not.
Maximizing the window with the context menu doesn't.

Maximizing the window from the system menu (Ctrl+Space) by selecting "Maximize" and pressing the Enter-key to confirm starts the conversation.
Trying the same using the access key ("x") does not.
Blocks: 955452
Attached patch PatchSplinter Review
*** Original post on bio 2042 as attmnt 2710 at 2013-08-14 01:03:00 UTC ***

This changes the mousedown handler in newtab-item to click, which ensures that the handler is activated only when both the mouseup and mousedown of the click were targeted at the item in question.

It moves the keyup handling code in the newtab binding to the keydown handler for similar reasons. This also matches the behavior in Firefox's awesomebar.
Attachment #8354479 - Flags: review?(benediktp)
Assignee: nobody → nhnt11
Status: NEW → ASSIGNED
*** Original post on bio 2042 at 2013-08-14 01:06:08 UTC ***

I forgot to mention I took the opportunity to remove the unnecessary if (!"Conversations" in window) check, since imWindows.jsm is imported when the window loads in instantbird.js. If it isn't ok to do that in this bug, I'll upload a new patch.
Comment on attachment 8354479 [details] [diff] [review]
Patch

*** Original change on bio 2042 attmnt 2710 at 2013-09-10 14:53:28 UTC ***

This fixes the reported bug and looks good! Thank you!
Attachment #8354479 - Flags: review?(benediktp) → review+
Whiteboard: [checkin-needed]
*** Original post on bio 2042 at 2013-09-12 22:39:45 UTC ***

attachment 8354479 [details] [diff] [review] (bio-attmnt 2710) doesn't apply, it's bitrotted because of bug 955503 (bio 2066). I've done a best effort update of it for the check-in. Please verify in the next nightly that things work as expected, thanks!
*** Original post on bio 2042 at 2013-09-12 23:05:14 UTC ***

http://hg.instantbird.org/instantbird/rev/d1fbb780a191
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [checkin-needed]
Target Milestone: --- → 1.5
You need to log in before you can comment on or make changes to this bug.