Skip to content

Commit

Permalink
Deprecate ‘in list’ variant of Link
Browse files Browse the repository at this point in the history
  • Loading branch information
VincentSmedinga committed Jan 21, 2024
1 parent 5d9942c commit 783a18d
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 24 deletions.
2 changes: 1 addition & 1 deletion packages/css/src/components/link/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ Geef een externe link altijd `rel="external"` mee. Vermijd `target="_blank"`, ma

### Bezochte stijl

Bezochte koppelingen geven aan dat een gebruiker de koppeling al heeft geopend. Gebruikte stijlen raden we af bij koppelingen, omdat ze de pagina vaak onoverzichtelijk maken. Het maakt navigeren door een pagina moeilijker. De inline koppeling heeft wel een bezochte stijl. Het vormt geen onderdeel van navigatie elementen die veel gescand worden.
Bezochte koppelingen geven aan dat een gebruiker de koppeling al heeft geopend. Gebruikte stijlen raden we af bij koppelingen, omdat ze de pagina vaak onoverzichtelijk maken. Het maakt navigeren door een pagina moeilijker. De inline koppeling heeft wel een bezochte stijl. Het vormt geen onderdeel van navigatie-elementen die veel gescand worden.

## Relevante WCAG eisen

Expand Down
4 changes: 3 additions & 1 deletion packages/react/src/Link/Link.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@ import type { AnchorHTMLAttributes, ForwardedRef, PropsWithChildren } from 'reac
import { Icon } from '../Icon/Icon'

type LinkOnBackground = 'default' | 'light' | 'dark'
type LinkVariant = 'standalone' | 'inline' | 'inList'
/** @deprecated Use `LinkList` instead. */
type DeprecatedLinkVariantInList = 'inList'
type LinkVariant = 'standalone' | 'inline' | DeprecatedLinkVariantInList

interface CommonProps extends Omit<AnchorHTMLAttributes<HTMLAnchorElement>, 'placeholder'> {
variant?: LinkVariant
Expand Down
9 changes: 0 additions & 9 deletions storybook/storybook-react/src/Link/Link.docs.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,6 @@ Gebruik geen overdaad aan inline-links op dezelfde pagina. Het verwart de gebrui

<Canvas of={LinkStories.Inline} />

### In List

Voor lijsten van koppelingen staan chevrons. De chevrons kunnen vervangen worden door andere iconen binnen een categorie.
Een combinatie van chevrons en iconen binnen een categorie is niet toegestaan.

<Canvas of={LinkStories.InList} />

<Canvas of={LinkStories.InListWithIcon} />

### On Dark Background

<Canvas of={LinkStories.onDarkBackground} />
Expand Down
13 changes: 0 additions & 13 deletions storybook/storybook-react/src/Link/Link.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -91,19 +91,6 @@ export const Inline: Story = {
],
}

export const InList: Story = {
args: {
variant: 'inList',
},
}

export const InListWithIcon: Story = {
args: {
variant: 'inList',
icon: Icons.EmailIcon,
},
}

export const onDarkBackground: Story = {
args: {
onBackground: 'dark',
Expand Down

0 comments on commit 783a18d

Please sign in to comment.