feat(NotificationBadge): Add animation for notify

[mfrances17]

What: Closes #10624

[patternfly-build]

Preview: https://patternfly-react-pr-11675.surge.sh

A11y report: https://patternfly-react-pr-11675-a11y.surge.sh

[thatblindgeye]

Met with Kayla quickly this morning to confirm expectation for when animations should be triggered. What you have here is essentially the expectation, i.e. we don't bake any specific logic internally, and instead the consumer has to handle logic when setting the hasAnimation prop. What I had done was whip up a quick example where hasAnimation={count % 5 === 0} so that the pf-m-notify class was applied or removed based on the count (no rhyme or reason to this, I just arbitrarily chose that condition).

Just some quick comments below based on the convo she and I had and what I noticed. I think this would be good to add any tests + example and pull out of draft.

[kaylachumley]

the animation itself looks good, just doubling down on the request for a reset animation button and a start or trigger animation button for example purposes(button could reset count to zero if its been triggered a bunch)

[thatblindgeye]

😍 this is looking nice, just one quick update below otherwise this is lgtm

[srambach]

Personally I'd put a button that just increments the count and has it notify each time, maybe a button to reset the count but that seems less important. I just sort of would like to be able to see that you can notify multiple times and having the count appear and disappear because it goes between zero and non-zero sort of obscures that. Also, because the buttons all shift when the count appears, it feels more obtrusive than it really is. But I can be convinced to leave it as is if others are happy.

[kmcfaul]

Code lgtm. +1 Sarah's suggestion with the demo but I think it can be done in a follow up if we want to get this PR in faster.

[srambach]

Beautiful tintinnabulation! 🔔