-
Notifications
You must be signed in to change notification settings - Fork 119
Timeline interactive #4336
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
Timeline interactive #4336
Conversation
|
🦋 Changeset detectedLatest commit: 8845a8a 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 |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
View your CI Pipeline Execution ↗ for commit 8845a8a.
☁️ Nx Cloud last updated this comment at |
View your CI Pipeline Execution ↗ for commit fd2ec1b.
☁️ Nx Cloud last updated this comment at |
Size Change: +182 B (0%) Total Size: 1.13 MB
ℹ️ View Unchanged
|
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 8845a8a:
|
eeaa714
to
a6be91e
Compare
Paste
|
Project |
Paste
|
Branch Review |
timeline-interactive
|
Run status |
|
Run duration | 04m 02s |
Commit |
|
Committer | PixeledCode |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
0
|
|
0
|
|
0
|
|
69
|
View all changes introduced in this branch ↗︎ |
TODO: I didn't specify it in the task, but can we add a section with example in Timeline docs for this, add it under Timeline first example Timeline with interactive titlesUse interactive titles only when the Timeline Event links to a specific moment in another view or surface. [warning callout] These titles should be used only when the event exists elsewhere and the user needs to be redirected to that exact point in time. If the event doesn't have a corresponding view, use a non-interactive title instead. |
TODO: Also for docs under Accessibility update the first bullet to read like this:
|
eb20d9a
to
8845a8a
Compare
<ListItem> | ||
The toggle button when using the `collapsible` prop: `collapsible` TimelineItems use the <Anchor href="/components/summary-detail">Summary Detail</Anchor> component. The toggle button is labeled by the value of the `timestamp` prop. If the value is too verbose for a screen reader or not a clear label for the collapsed content, provide a helpful label for the button using the collapsibleLabelText prop on TimelineItem. | ||
</ListItem> |
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.
👏
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!
Paste
|
Project |
Paste
|
Branch Review |
timeline-interactive
|
Run status |
|
Run duration | 07m 32s |
Commit |
|
Committer | Shoaib Ahmed |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
0
|
|
0
|
|
0
|
|
125
|
View all changes introduced in this branch ↗︎ |
https://paste-docs-git-timeline-interactive-twilio.vercel.app