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

[RFC] Removed DWM NCR attributes, added WindowChrome. Fixes #481, #488, #496. #497

Closed
wants to merge 1 commit into from

Conversation

bitterskittles
Copy link
Contributor

No description provided.

… DWM non-client rendering directly.

Added http://archive.msdn.microsoft.com/WPFShell for MA.M .NET 4 client profile.
Fixes #481, #488, #496.
@AzureKitsune
Copy link
Member

I like it. However, that extra dependency for .NET 4. Hmm. Well, with nuget, I'm fine with it.

I am neutral on this. I wonder what @flagbug , @shiftkey and @punker76 think.

@spiritdead
Copy link
Contributor

well i tested you fork, i can run my windowsformhost :S

@shiftkey
Copy link

shiftkey commented Jun 1, 2013

Let me review this and see if we can do it without taking a dependency on Microsoft.Windows.Shell

@ghost ghost assigned shiftkey Jun 1, 2013
@flagbug
Copy link
Member

flagbug commented Jun 1, 2013

There is a NuGet package, we could use that as dependency for a NET40 Mahapps.Metro package

@AzureKitsune
Copy link
Member

@flagbug Yeah, I use the NuGet package. I was worried about it being a dependency for people not using the NuGet package.

@shiftkey
Copy link

shiftkey commented Jun 2, 2013

So the source for this is distributed under MS-PL (the same license we use) so we could pull in just that class and avoid the whole "dependency" headaches.

Stay tuned, will post more here and ensure everyone is happy with it.

@shiftkey
Copy link

shiftkey commented Jun 2, 2013

#503 has this change and other patches so that it is ready to publish - just a couple of minor things to sort out first.

@shiftkey shiftkey closed this Jun 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants