-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add snapshots #424
Comments
Actually, the main idea of Snapgitdist is to show snapshot in Styleguide. It does not allow to run automated tests or snapshot comparisons, and cannot be set up to be used for Continuous Integration. See styleguidist/snapguidist#16 ans styleguidist/react-styleguidist#877 |
Adding snapshots could still be useful no ? |
@ptbrowne You mean, in styleguidist ? |
No, in general, to components |
related #452 |
Is this issue still relevent since #452 has been done & merged? |
I think we can close |
Check out snapguidist to see if it could be useful.
Add snapshots to existing components
ActionMenu⚠️ Not shown in styleguidist so not tested properly
Alerter
Avatar
Badge
Button
ButtonAction
Checkbox
Field
Hero
I18n
Icon
Input
IntentHeader
IntentIframe
IntentModal
IntentOpener
Label
ListItemText
Media
Menu
MidEllipsis
Modal
Nav
Overlay
Panel
Radio
SelectBox
SelectionBar
Sidebar
Spinner
Tabs
Text
Textarea
Toggle
The text was updated successfully, but these errors were encountered: