Adds Storybook test cases for scene-ui.js #6564
Draft
+101
−12
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What?
Adds Storybook test cases for scene-ui.js
Also rounds out a few other Storybook stories.
Why?
How to test
npm run storybook
http://localhost:6006/?path=/story/scene-ui--base
Limitations
None of the Mobile test cases for Tooltip worked before, and they still don't. The UI in the actual app is fine. It's not clear why Storybook gives bad results here.
Alternatives considered
If the Mobile test cases for Tooltip were included, it would be natural for a dev to conclude the actual UI is wrong. That would waste dev time on something that isn't broken in the actual app.
Open questions
How could Storybook be better configured, or the Tooltip story better written?
Additional details or related context
This depends on Matt's Storybook fixes. #6563