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

Fix default elevation of Card Flipper #2801

Merged

Conversation

nicolaihenriksen
Copy link
Contributor

Partial fix for #2729

This fixes the broken default elevation of the (Card) Flipper. It was a simple matter of a default elevation not being set on the MaterialDesignCardFlipper style.

I added a UI test to ensure this default is applied, but it fails because of the static constructor in FlipperAssist. I was unable to get this to work and may need some input on how to get it to locate the resource with the Uri when executing in the MaterialDesignThemes.UITests context.

nicolaihenriksen and others added 2 commits July 28, 2022 17:17
The UI test fails because of the static ctor in FlipperAssist. I was unable to get this to work and may need some input on how to get it to locate the resource with the Uri when executing in the MaterialDesignThemes.UITests context.
@Keboo Keboo added this to the 4.6.0 milestone Jul 29, 2022
@Keboo Keboo merged commit 1f4a020 into MaterialDesignInXAML:master Jul 29, 2022
@Keboo
Copy link
Member

Keboo commented Jul 29, 2022

Thank you. I fixed the issue by moving out the static loading of the resources into a separate class. This effectively defers loading of the resources until GetDropShadow is invoked

@nicolaihenriksen nicolaihenriksen deleted the fixCardFlipperElevation branch August 1, 2022 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants