Initial implementation of modal flyouts.

[grokys]

Trying again... I hate Git sometimes!

[AzureKitsune]

Hmmm, I don't my VS with me at the moment. How is that Modal grid any different from the overlay used by Dialogs? Also, you have to add it to the CleanWindow style as well.

[grokys]

The dialog overlay is on top of FlyoutsControl so it disables the Flyouts as well as the main window content . I assume this is desirable behaviour.

If you think I'm on the right track I can add it to CleanWindow too.

[AzureKitsune]

Yes, you are on the right track. CleanWindow needs to have this before the PR can be merged.

[grokys]

Just took a look at adding this to the CleanWindow style but it appears that Flyouts don't currently work with CleanWindow; if I apply the CleanWindowStyle to the FlyoutDemo nothing happens when I click a button...

[flagbug]

Nice! Just one thing, can you use the Grid here for creating the overlay effect, or if thats not possible, respect the current Theme for the overlay color?

[grokys]

Hi @flagbug, as discussed about the currently existing overlay can't be used as it's on top of FlyoutsControl. I can make it respect the theme; what color should I use? BlackColorBrush as for the existing modal overlay?

[flagbug]

@grokys Yup

[AzureKitsune]

@grokys Once you replicate your changes in MetroWindow.xaml into CleanWindow.xaml, we can merge this. 😄

[grokys]

#835 is holding me up on this.

[AzureKitsune]

@grokys Done. See #840

[grokys]

This should now be implemented.

[AzureKitsune]

Oh man, the merge broke something. We won't be able to merge this "nicely". I'll see about manually merging this tonight.

[grokys]

Hmm, this seems to happen every time I fork a repo. Wonder what I'm doing wrong?

[AzureKitsune]

Usually, it is better to rebase instead of merge.

[grokys]

Yeah, that's what I did...

[flagbug]

Merged by hand, thanks @grokys !

[grokys]

Thanks a million @flagbug - was about to do that myself this evening!