fix(carousel): removed flickering

[agliga]

Description

• Before typescript was introduced, the left/right offsets for each items could be initially undefined. This was to verify that the component was being re-rendered. After typescript, the offsets were set to 0. There was a check to verify if the carousel is animating which would specifically check for undefined. Since these were always set, that check would never pass.
• We need an initial value of null/undefined. The problem with setting 0, is that 0 could be a possible offset and cannot be used to check if there is a re-render.
• One more issue: there are some comparison checks. If you do any number > or < to undefined, it evaluates as false. This is not the case if its 0. Unfortunately, the code was written expecting those checks to evaluate as false.

Context

• I added undefined as the default value.
• I added ! to the undefined values to allow typescript to pass compilation.
• I also added an example of how to do controlled carousel.

References

#2206

[changeset-bot]

🦋 Changeset detected

Latest commit: 80febe5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@ebay/ebayui-core	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR