[Collapsible][Accordion] Fix initial animation playback in Firefox and Safari

[andy-hook]

closes #1265

After some exploration it dawned that perhaps these browsers need a little bump to replay the animation from scratch for the initial size retrieval.

I'll try and validate this behaviour in a smaller example but this does fix the issue.

@benoitgrelard Do you remember why we moved away from the clearing the shorthand in the first place? I remember that had problems but i'm questioning if removing the name property alone is all we need.

[benoitgrelard]

@benoitgrelard Do you remember why we moved away from the clearing the shorthand in the first place? I remember that had problems but i'm questioning if removing the name property alone is all we need.

It seems because we had the issue in #1040 so maybe run that scenario and see if your new change creates that regression?

[benoitgrelard]

Approved pending check in other comment 👍

[andy-hook]

maybe run that scenario and see if your new change creates that regression

Looks good.

i'm questioning if removing the name property alone is all we need.

Tested in all browsers + IOS and still looks good, let me know if you're good with this additional change.

[benoitgrelard]

I'm wondering if we could add some cypress tests or chromatic tests to test all these behaviours, although not sure how feasible that is given the tight criterias with animation.

[andy-hook]

I'm wondering if we could add some cypress tests or chromatic tests to test all these behaviours, although not sure how feasible that is given the tight criterias with animation.

This would be ideal though feasibility is definitely an issue given the real time nature. Browser differences make this even harder 😞