[Mono-bugs] [Bug 429153] ListView: WndProc Mouse Events

[bugzilla_noreply at novell.com]

https://bugzilla.novell.com/show_bug.cgi?id=429153

User johnm at hlaustralia.com.au added comment
https://bugzilla.novell.com/show_bug.cgi?id=429153#c1





--- Comment #1 from John Mortlock <johnm at hlaustralia.com.au>  2008-09-24 22:35:05 MDT ---
Created an attachment (id=241543)
 --> (https://bugzilla.novell.com/attachment.cgi?id=241543)
Proposed Fix

This is my attempt at a fix for this.
Basically this is what i have done, I'm sure there are better ways!

1. Redirect WndProc messages from the internal control back to the owner. It
still handles events it needs to know about WM_PAINT for example.

2. the listview WndProc than handles the required mouse events and passes them
down to the item_control, i had to add some guards to block recursion errors as
I did not want to mess with the ItemsMouseXXX functions too much.

3. Remove some of the owner.MouseXXX events in the item control to stop
duplicate events from being fired.

4. ItemsMouseUp can set the selection (if not already set) to do this I moved
the logic in ItemsMouseDown into a function and called that.

With the patch the test case i originally submitted works, just wonder what
i've broken :)


-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.