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

Introduce restore and minimize animations #1514

Merged
merged 2 commits into from
Aug 24, 2014
Merged

Introduce restore and minimize animations #1514

merged 2 commits into from
Aug 24, 2014

Conversation

davidzech
Copy link
Contributor

Achieves this by avoiding usage of WindowStyle None unless one or more of these options are true:
UseNoneWindowStyle
IgnoreTaskBarOnMaximize

If either of these are true, Minimize/Restore animations are disabled. It simply isn't compatible with WindowStyleNone.

GlowWindows are restored after 200ms of window being restored, like VS2013 does.

An IgnoreTaskBarOnMaximize toggle has been added to the MetroDemo, which offsers to serve a quick way to both toggle this flag and minimize/restore animations.

@mrahhal
Copy link
Contributor

mrahhal commented Aug 22, 2014

@AdditionalPylons a git reset 8c589a86076bf8196b8fadfce1dbb81619374be5 followed by a git commit -m "Fix null access when glowwindows are null" would clean the history (would keep the first commit as it is, and squash the 3rd and the 4th commit with the 2nd). This benefits the master branch when merged.

If in doubt you can try it with a new branch. Just make sure that you're in a clean working directory.

@davidzech
Copy link
Contributor Author

Ok, thanks for the tip.

@flagbug flagbug added this to the v1.0 milestone Aug 23, 2014
flagbug added a commit that referenced this pull request Aug 24, 2014
…tions

Introduce restore and minimize animations
@flagbug flagbug merged commit d57d1e4 into MahApps:master Aug 24, 2014
@flagbug
Copy link
Member

flagbug commented Aug 24, 2014

Looks good, thanks!

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.

3 participants