Skip to content

Commit

Permalink
Removed work in progess note and new grammarly fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
Niko Helle committed Sep 14, 2022
1 parent c614aa2 commit 55e1ea9
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 14 deletions.
16 changes: 8 additions & 8 deletions site/src/docs/components/cookie-consent/api.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -63,14 +63,14 @@ Cookies for domains on the public suffix list cannot accept subdomain cookies. E

##### Cookie groups

| Property | Description |
| ----------------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| `group[].groupId` / `group[].commonGroup` | Both optional. If a common group is used, then the `commonGroup`. If this is a custom group provide an `groupId`. Not required, if there are no cookies that have `groupId` set. |
| `group[].title` | Only for custom groups. Title of the group. Cannot be set for a common group. **Must be unique.** |
| `group[].text` | Only for custom groups. Cannot be set for a common group. Description of the group. |
| `group[].expandAriaLabel` | Only for custom groups. Cannot be set for a common group. Aria-label for the button to show more details of the group. |
| `group[].checkboxAriaDescription` | Only for custom groups. Cannot be set for a common group. Aria-description for the checkbox for giving consent for all cookies in the group |
| `group[].cookies` | Optional list of cookies in this group. |
| Property | Description |
| ----------------------------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| `group[].groupId` / `group[].commonGroup` | Both optional. If a common group is used, then the `commonGroup`. If this is a custom group provide an `groupId`. Not required, if there are no cookies that have a `groupId` set. |
| `group[].title` | Only for custom groups. Title of the group. Cannot be set for a common group. **Must be unique.** |
| `group[].text` | Only for custom groups. Cannot be set for a common group. Description of the group. |
| `group[].expandAriaLabel` | Only for custom groups. Cannot be set for a common group. Aria-label for the button to show more details of the group. |
| `group[].checkboxAriaDescription` | Only for custom groups. Cannot be set for a common group. Aria-description for the checkbox for giving consent for all cookies in the group |
| `group[].cookies` | Optional list of cookies in this group. |
| [Table 3:Cookie group properties] |

##### Cookie information
Expand Down
2 changes: 1 addition & 1 deletion site/src/docs/components/cookie-consent/index.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ export default ({ children, pageContext }) => <TabsLayout pageContext={pageConte

## Usage

This documentation page is about HDS CookieCompliance component. If you are looking for documentation about cookies in general, please refer to [HDS cookie pattern documentation page](/patterns/cookies/basics).
This documentation page is about HDS CookieCompliance component. If you are looking for documentation about cookies in general, please refer to the [HDS cookie pattern documentation page](/patterns/cookies/basics).

### Example

Expand Down
5 changes: 0 additions & 5 deletions site/src/docs/components/cookie-consent/tabs.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,6 @@ import PageTabs from '../../../components/PageTabs';
application for the first time.
</LeadParagraph>

<Notification label="The component is work in progress!" type="alert" className="siteNotification">
This component is work in progress. Breaking changes are possible and the final accessibility audit has not been done.
It has been released for early users and to gather feedback.
</Notification>

<PageTabs pageContext={props.pageContext}>
<PageTabs.TabList>
<PageTabs.Tab href="/">Usage</PageTabs.Tab>
Expand Down

0 comments on commit 55e1ea9

Please sign in to comment.