MudCollapse: Rework animation

[danielchalmers]

Description

Fixes #8939 by reworking the animation entirely to use CSS transitions on a grid display as opposed to calculating the height through JavaScript. This should be more performant and stable.

Impacts other components like the Expansion Panels, Tree View, Nav Menu. I'd like it if someone with more experience with those components can try the new transition effect locally.
 

How Has This Been Tested?

visually

Type of Changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation (fix or improvement to the website or code docs)

Before: #8939 (comment)

After:

video6.mp4

Animation is now a static 300ms which is between 50% slower and 333% faster, depending on how tall the content was before.

CollapseState, _state, AnimationEndAsync are now private but I don't believe I made any other breaking changes aside from rewriting the CSS obviously (but the CSS classes themselves are still the same).

Checklist

• The PR is submitted to the correct branch (dev).
• My code follows the code style of this project.
• I've added relevant tests.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.04%. Comparing base (6ac51c8) to head (c11de65).
Report is 7 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev   #10056      +/-   ##
==========================================
- Coverage   91.10%   91.04%   -0.06%     
==========================================
  Files         410      410              
  Lines       12476    12467       -9     
  Branches     2421     2420       -1     
==========================================
- Hits        11366    11351      -15     
- Misses        565      572       +7     
+ Partials      545      544       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ScarletKuro]

LGTM, I absolutely love this.

[ScarletKuro]

cc @henon @jperson2000 @igotinfected if anyone wants to test anything visually, feel free
Otherwise want to get this asap :)

[henon]

Awesome how much code could be removed by this change!

[ScarletKuro]

Added to the v8 migration guide at #9953

[oscarsjocrona]

Great! Thank you

[bsallesp]

Hi folks, migrating from 7.16 to 8.3, my MudCollapse components didn't expand/collapse anymore. Any thoughts?

[henon]

@bsallesp See if it works in an incognito tab. if so this is a browser cache issue.

[Anu6is]

It works as expected in the docs - https://mudblazor.com/components/collapse#api
try clearing your cache or reproduce a TryMud example where it does not work

[bsallesp]

on it

[bsallesp]

You guys saved the day. Let me know If a can help with something else. I'm open for participating for new communities. See ya!

[henon]

Let me know If a can help with something else. I'm open for participating for new communities.

We appreciate any help we can get. Whenever you see something that needs improving, even if it's just small corrections/improvements in the docs, just open a PR for it. Every little bit helps!

[bsallesp]

Noy urgent, but, I'm trying to clean my browser cache (brave), specially things from Blazor 7. I accept suggestions.

In meanwhile, using anonymous tab.