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
Pog, Toast, Tooltip, Upsell, Video: add visual diff testing and screenshot to description in VSCode (Flow extension) #2057
Conversation
4765c22
to
6c51203
Compare
3cd4727
to
cd5a0bf
Compare
✅ Deploy Preview for gestalt ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
cd5a0bf
to
87c32d5
Compare
@@ -0,0 +1,15 @@ | |||
describe('Pog visual dark mode regression check', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should these files be Flow typed? Have we been add Flow annotations to the other visual test files?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have we been add Flow annotations to the other visual test files?
NO
packages/gestalt/src/Pog.js
Outdated
@@ -89,7 +89,11 @@ type Props = {| | |||
/** | |||
* [Pog](https://gestalt.pinterest.systems/pog) is a lower-level functional component to show the active, hovered, & focused states of [IconButton](https://gestalt.pinterest.systems/pog/iconbutton). | |||
* | |||
*This abstraction to allow for links that look like IconButton. | |||
* This abstraction to allow for links that look like IconButton. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(pre-existing) typo: "This is an abstraction to allow for…"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DONE
@@ -40,6 +40,10 @@ type Props = {| | |||
* [Toasts](https://gestalt.pinterest.systems/toast) can educate people on the content of the screen, provide confirmation when people complete an action, or simply communicate a short message. | |||
* | |||
* Toast is purely visual. In order to properly handle the showing and dismissing of Toasts, as well as any animations, you will need to implement a Toast manager. | |||
* | |||
* ⚠️ Please note: Toast is not currently supported in dark mode. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙏
packages/gestalt/src/Upsell.js
Outdated
@@ -131,7 +131,9 @@ type Props = {| | |||
/** | |||
* [Upsells](https://gestalt.pinterest.systems/upsell) are banners that display short messages that focus on promoting an action or upgrading something the user already has. | |||
* | |||
* ⚠️ Please note: Upsell is not currently supported in dark mode. | |||
* ⚠️ Please note: Upsells is not currently supported in dark mode. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: Upsell should be singular
87c32d5
to
055c81b
Compare
055c81b
to
e1f1493
Compare
Summary
What changed?
Pog, Toast, Tooltip, Upsell, Video: add visual diff testing and screenshot to description in VSCode (Flow extension)
Why?
Provide basic visual testing coverage for most Gestalt components and link screenshots to VSCode's Flow extension that shows component screenshot below the component description on hover for better in-editor documentation.
OKR: