-
Notifications
You must be signed in to change notification settings - Fork 119
feat(website): ai experience #3994
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: c73e61e The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
☁️ Nx Cloud ReportCI is running/has finished running commands for commit c73e61e. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit c73e61e:
|
Size Change: +1 B (0%) Total Size: 1.08 MB
ℹ️ View Unchanged
|
Passing run #8335 ↗︎Details:
Review all test suite changes for PR #3994 ↗︎ |
packages/paste-website/src/pages/experiences/navigation/index.mdx
Outdated
Show resolved
Hide resolved
packages/paste-website/src/pages/experiences/artificial-intelligence/index.mdx
Outdated
Show resolved
Hide resolved
packages/paste-website/src/pages/experiences/artificial-intelligence/index.mdx
Outdated
Show resolved
Hide resolved
packages/paste-website/src/pages/experiences/artificial-intelligence/index.mdx
Outdated
Show resolved
Hide resolved
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.
Overall really great! Left some comments. Think the changes for .cypress vrt are missing too
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.
LGTM! Great work
packages/paste-website/src/pages/experiences/artificial-intelligence/index.mdx
Show resolved
Hide resolved
Passing run #8334 ↗︎Details:
Review all test suite changes for PR #3994 ↗︎ |
With AI-driven products, there can be a temptation to communicate the “newness” or “magic” of the system’s predictions through its UI metaphors. | ||
However, unfamiliar UI touchpoints can make it harder for users to learn to use your system, potentially leading to degraded understanding of, or trust in, your product…Instead, anchor new users with familiar UI patterns and features. | ||
|
||
—[People + AI Guidebook](https://pair.withgoogle.com/guidebook/patterns#anchor-on-familiarity), Google |
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.
fix: There's an example on the Switch docs on how to get the source to be styled as intended: https://github.com/twilio-labs/paste/blob/main/packages/paste-website/src/pages/components/switch/index.mdx
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.
definitely forgot that was a thing ty
packages/paste-website/src/pages/experiences/navigation/index.mdx
Outdated
Show resolved
Hide resolved
This PR currently has a merge conflict. Please resolve this and then re-add the |
0666b8c
to
b858c78
Compare
Experiences