Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SplitButton does not mark mouse events handled #2545

Closed
bloritsch opened this issue Jun 13, 2016 · 5 comments
Closed

SplitButton does not mark mouse events handled #2545

bloritsch opened this issue Jun 13, 2016 · 5 comments
Milestone

Comments

@bloritsch
Copy link

What steps will reproduce this issue?

We have an application that has a canvas that allows users to add elements by mouse click. We had some formatting controls accessible via a SplitButton. When a user opened the drop down, the mouse was also over the canvas. When they selected an option, the mouse click was bubbled through to our canvas and our code attempted to create an element at that position.

Expected outcome

We expected the SplitButton drop down to behave more like a ComboBox. When the user selects an option from the ComboBox, the mouse up event is marked as e.Handled = true. That prevents the erroneous click on the canvas.

Our short term fix for now is to use a ComboBox and a separate button.

Environment

  • MahApps.Metro v1.1.2
  • Windows 7 pro
  • Visual Studio 2013
  • .NET Framework 4.5
@punker76
Copy link
Member

punker76 commented Jul 8, 2016

@bloritsch Please test this again after merged PR #2572 thx!

@punker76
Copy link
Member

@bloritsch Could you retest this with latest pre-release?

@bloritsch
Copy link
Author

I'm sorry, I haven't gotten back to you yet. I've been so crazy busy with
this software release and then a bunch of busy stuff at home. Hopefully I
will be able to test it tomorrow. Is the fix in a Pre-release or the
official release?

On Thu, Jul 21, 2016 at 3:54 PM, Jan Karger notifications@github.com
wrote:

@bloritsch https://github.com/bloritsch Could you retest this with
latest pre-release?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#2545 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAL4DmSuXzyt8gqL6RfOUUa6GijqMp8jks5qX86SgaJpZM4I0TKD
.

@punker76
Copy link
Member

@bloritsch np, try latest pre-release, thx.

@bloritsch
Copy link
Author

Confirmed. It's working now.

@punker76 punker76 added this to the 1.3.0 milestone Jul 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants