-
Notifications
You must be signed in to change notification settings - Fork 375
Update broken links. #9310
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
Update broken links. #9310
Changes from all commits
d62a6f3
51bfb49
7c16669
12b6059
015d94b
89491eb
e06c6ca
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -137,7 +137,7 @@ const AlertTimeout: React.FunctionComponent = () => { | |
|
|
||
| An alert can contain additional, hidden information that is made visible when users click a caret icon. This information can be expanded and collapsed each time the icon is clicked. | ||
|
|
||
| It is not recommended to use an expandable alert with a `timeout` in a toast [alert group](/components/alert-group) because the alert could timeout before users have time to interact with and view the entire alert. | ||
| It is not recommended to use an expandable alert with a `timeout` in a [toast alert group](/components/alert#toast-alert-group) because the alert could timeout before users have time to interact with and view the entire alert. | ||
|
|
||
| See the [toast alert considerations](/components/alert/accessibility#toast-alerts) section of the alert accessibility documentation to understand the accessibility risks associated with using toast alerts. | ||
|
|
||
|
|
@@ -354,7 +354,7 @@ import { Alert, AlertActionCloseButton } from '@patternfly/react-core'; | |
|
|
||
| ### Dynamic live region alerts | ||
|
|
||
| Alerts that are asynchronously appended into dynamic [alert groups](/components/alert-group) via the `isLiveRegion` property will be announced to assistive technology the moment the change happens, following the strategy used for `aria-atomic`, which defaults to false. This means only changes of type "addition" will be announced. | ||
| Alerts that are asynchronously appended into dynamic [alert groups](/components/alert/#alert-group-examples) via the `isLiveRegion` property will be announced to assistive technology the moment the change happens, following the strategy used for `aria-atomic`, which defaults to false. This means only changes of type "addition" will be announced. | ||
|
|
||
| ```ts file="AlertDynamicLiveRegion.tsx" | ||
| ``` | ||
|
|
@@ -379,7 +379,7 @@ Alert groups can be one of the following variants: | |
| | Toast | Toast alert groups contain alerts that typically appear in response to an asynchronous event or user action. These groups are positioned on top of other content at the top right of the page.| | ||
| | Dynamic | Dynamic alert groups contain alerts that typically appear in response to a user action, and are seen within the normal page content flow.| | ||
|
|
||
| Dynamic alerts that are generated after the page initially loads must be appended to either a toast or dynamic `AlertGroup`, both of which must use the `isLiveRegion` property. New alerts appended to a toast or dynamic group will be announced by assistive technologies the moment the change happens. For information about customizing this announcement, read the [aria-atomic and aria-relevant](/components/alert-group/accessibility#aria-atomic-and-aria-relevant) section of the alert group accessibility documentation. | ||
| Dynamic alerts that are generated after the page initially loads must be appended to either a toast or dynamic `AlertGroup`, both of which must use the `isLiveRegion` property. New alerts appended to a toast or dynamic group will be announced by assistive technologies the moment the change happens. For information about customizing this announcement, read the [aria-atomic and aria-relevant](/components/alert/accessibility#aria-atomic-and-aria-relevant) section of the alert accessibility documentation. | ||
|
|
||
| ### Static inline alert group | ||
|
|
||
|
|
@@ -414,7 +414,7 @@ Users navigating via keyboard or another assistive technology will need a way to | |
|
|
||
| The following example shows how alerts can be triggered by an asynchronous event in the application. You can customize how an alert will be announced to assistive technology by adjusting the value of the `aria-live` property. Click the "start async" alert button below and then click the buttons in the above toast examples to demonstrate how asynchronous events add alerts to a group. Click the "stop async alerts" button to halt this behavior. | ||
|
|
||
| See the [alert group accessibility tab](/components/alert-group/accessibility) for more information on customizing this behavior. | ||
| See the [alert accessibility tab](/components/alert/accessibility) for more information on customizing this behavior. | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This link is still broken
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This link is correct - but the accessibility doc is not available in the react workspace. it's only available when react workspace is consumed in org and built along with the accessibility docs. |
||
|
|
||
| ```ts file="./AlertGroupAsync.tsx" | ||
| ``` | ||
|
|
||
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.
this link is not working as well
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.
I did some investigating - I believe the link itself is correct, it's just not autoscrolling to the linked example, even though it's updating the URL. I'm not sure why.