react-spectrum icon indicating copy to clipboard operation
react-spectrum copied to clipboard

CSF 3.0 Tooltip and TooltipTrigger

Open ktabors opened this issue 2 years ago โ€ข 11 comments

There was a code change!!!!!! TooltipTrigger didn't support shouldFlip or containerPadding. Both of those are now supported.

Part of the Storybook CSF 3.0 migration in issue 3056.

Migration included refactoring to use Storybook controls.

There are changes to Chromatic so we'll need to confirm the migrated stories didn't change and approve the new stories.

  • [ ] Run Chromatic and approve the new stories.

โœ… Pull Request Checklist:

  • [ ] Included link to corresponding React Spectrum GitHub Issue.
  • [ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • [ ] Filled out test instructions.
  • [ ] Updated documentation (if it already exists for this component).
  • [ ] Looked at the Accessibility Practices for this feature - Aria Practices

๐Ÿ“ Test Instructions:

Review the changes to Tooltip and TooltipTrigger in storybook. Confirm that the correct stories were migrated or dropped, and controls work and make sense.

๐Ÿงข Your Project:

RSP

ktabors avatar Jun 09 '22 23:06 ktabors

Gonna rerun chromatic to see if those previous runs were flakey test

LFDanLu avatar Jun 20 '22 23:06 LFDanLu

Here's chromatic for the change https://www.chromatic.com/build?appId=5f0dd5ad2b5fc10022a2e320&number=338

snowystinger avatar Aug 18 '22 04:08 snowystinger