-
Notifications
You must be signed in to change notification settings - Fork 79
Closed
Labels
4 - verifiedIssues that have been tested, confirmed as mitigated, and are ready to close.Issues that have been tested, confirmed as mitigated, and are ready to close.ArcGIS Map ViewerIssues logged by ArcGIS Map Viewer team members.Issues logged by ArcGIS Map Viewer team members.bugBug reports for broken functionality. Issues should include a reproduction of the bug.Bug reports for broken functionality. Issues should include a reproduction of the bug.calcite-componentsIssues specific to the @esri/calcite-components package.Issues specific to the @esri/calcite-components package.estimate - 1Very small fix or change (potentially a single line), doesn't require updates to tests.Very small fix or change (potentially a single line), doesn't require updates to tests.figma changesIssues that require additions or updates to the Figma UI Kit where no `design` label existsIssues that require additions or updates to the Figma UI Kit where no `design` label existsgood first issueIssues that can be worked on by contributors new to calcite-components.Issues that can be worked on by contributors new to calcite-components.i18n-l10nissues dealing with internationalization/localizationissues dealing with internationalization/localizationimpact - p2 - want for an upcoming milestoneUser set priority impact status of p2 - want for an upcoming milestoneUser set priority impact status of p2 - want for an upcoming milestonep - lowIssue is non core or affecting less that 10% of people using the libraryIssue is non core or affecting less that 10% of people using the library
Description
Check existing issues
- I have checked for existing issues to avoid duplicates
Actual Behavior
The underline animation does not correctly animate in the expected direction when in RTL.
Expected Behavior
I'd expect the animation to correctly display as it does in LTR, but flipped.
cc @SkyeSeitz @ashetland - I know we've talked about changing this animation altogether and favoring something less... busy - maybe an additional px height on hover, just color change, etc., so could be a good opportunity to look at that.
Related: #6569
Reproduction Sample
https://codepen.io/mac_and_cheese/pen/qBQXYKB?editors=1000
Reproduction Steps
- Go to Codepen
- See difference in animation
Reproduction Version
Latest
Relevant Info
No response
Regression?
No response
Priority impact
p2 - want for current milestone
Impact
No response
Calcite package
- @esri/calcite-components
- @esri/calcite-components-react
Esri team
ArcGIS Map Viewer
Metadata
Metadata
Labels
4 - verifiedIssues that have been tested, confirmed as mitigated, and are ready to close.Issues that have been tested, confirmed as mitigated, and are ready to close.ArcGIS Map ViewerIssues logged by ArcGIS Map Viewer team members.Issues logged by ArcGIS Map Viewer team members.bugBug reports for broken functionality. Issues should include a reproduction of the bug.Bug reports for broken functionality. Issues should include a reproduction of the bug.calcite-componentsIssues specific to the @esri/calcite-components package.Issues specific to the @esri/calcite-components package.estimate - 1Very small fix or change (potentially a single line), doesn't require updates to tests.Very small fix or change (potentially a single line), doesn't require updates to tests.figma changesIssues that require additions or updates to the Figma UI Kit where no `design` label existsIssues that require additions or updates to the Figma UI Kit where no `design` label existsgood first issueIssues that can be worked on by contributors new to calcite-components.Issues that can be worked on by contributors new to calcite-components.i18n-l10nissues dealing with internationalization/localizationissues dealing with internationalization/localizationimpact - p2 - want for an upcoming milestoneUser set priority impact status of p2 - want for an upcoming milestoneUser set priority impact status of p2 - want for an upcoming milestonep - lowIssue is non core or affecting less that 10% of people using the libraryIssue is non core or affecting less that 10% of people using the library