timheuer / callisto

A control toolkit for Windows 8 XAML applications. Contains some UI controls to make it easier to create Windows UI style apps for the Windows Store in accordance with Windows UI guidelines.
http://timheuer.com/blog/archive/2012/05/31/introducing-callisto-a-xaml-toolkit-for-metro-apps.aspx
Other
338 stars 108 forks source link

Null object reference error setting isopen property to true when PlacementTarget property is not set #226

Closed rprakashg closed 11 years ago

rprakashg commented 11 years ago

I noticed an issue with Flyout control; if you don't set the PlacementTarget property and set IsOpen to true, throws a null object reference error

timheuer commented 11 years ago

By design. The Flyout needs to know where it originates from.

rprakashg commented 11 years ago

Sure how about adding a constructor that takes PlacementTarget as parameter since its required for Flyout, this way will be forced to specify

timheuer commented 11 years ago

That or make the exception better :-)


http://timheuer.com/blog/ twitter: @timheuer

On Fri, Sep 27, 2013 at 12:49 AM, Ram Gopinathan notifications@github.comwrote:

Sure how about adding a constructor that takes PlacementTarget since its required for Flyout, this way will be forced to specify

— Reply to this email directly or view it on GitHubhttps://github.com/timheuer/callisto/issues/226#issuecomment-25228382 .

rprakashg commented 11 years ago

That would be good too :) Awesome toolkit BTW