Skip to content

Commit

Permalink
Add pattern-library documentation for Tab and TabList
Browse files Browse the repository at this point in the history
  • Loading branch information
lyzadanger committed Jan 27, 2023
1 parent 389c03c commit e647f6f
Show file tree
Hide file tree
Showing 2 changed files with 329 additions and 0 deletions.
322 changes: 322 additions & 0 deletions src/pattern-library/components/patterns/navigation/TabPage.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,322 @@
import classnames from 'classnames';
import { useState } from 'preact/hooks';

import { Card, Tab, TabList } from '../../../../next';
import Library from '../../Library';
import Next from '../../LibraryNext';

export default function TabPage() {
const [prefPanel, setPrefPanel] = useState('notifications');
const [sidebarPanel, setSidebarPanel] = useState('annotations');
const [sidebarPanel2, setSidebarPanel2] = useState('annotations');
const [sidebarPanel3, setSidebarPanel3] = useState('annotations');
return (
<Library.Page
title="Tabs"
intro={
<p>
<code>Tab</code> and <code>TabList</code> are presentational
components for rendering accessible tabs.
</p>
}
>
<Library.Section
title="Tab"
intro={
<p>
<code>Tab</code> generates a button with appropriate ARIA
attributes.
</p>
}
>
<Library.Pattern title="Status">
<p>
<code>Tab</code> is a new component.
</p>
</Library.Pattern>

<Library.Pattern title="Usage">
<Next.Usage componentName="Tab" />
<Library.Example>
<Library.Demo title="Basic (non-interactive) example" withSource>
<div role="tablist" className="gap-x-6 flex">
<Tab>
Annotations
<span className="relative bottom-[3px] left-[2px] text-[10px]">
{52}
</span>
</Tab>
<Tab selected>
Page Notes
<span className="relative bottom-[3px] left-[2px] text-[10px]">
{4}
</span>
</Tab>
<Tab>
Orphans
<span className="relative bottom-[3px] left-[2px] text-[10px]">
{2}
</span>
</Tab>
</div>
</Library.Demo>
</Library.Example>

<ul>
<li>
<code>Tab</code>s <em>must</em> be direct children of an element
with <code>role={'"tablist"'}</code> (or use the{' '}
<code>TabList</code> component).
</li>
<li>
You <em>should</em> provide an{' '}
<a href="https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-controls">
<code>aria-controls</code>
</a>{' '}
attribute to each <code>Tab</code>. This is not always feasible in
our applications.
</li>
</ul>
</Library.Pattern>

<Library.Pattern title="Props">
<Library.Example title="aria-controls">
An element with <code>{'role="tab"'}</code> should set an{' '}
<code>aria-controls</code> attribute when possible. See the full{' '}
<code>TabList</code> example below.
</Library.Example>

<Library.Example title="selected">
This boolean property asserts that the <code>Tab</code> is currently
selected and the <code>tabpanel</code> it controls (where relevant)
is active and visible.
</Library.Example>

<Library.Example title="textContent">
<p>
<code>textContent</code> is a string representing the text content
of the tab when selected.{' '}
<strong>
Setting <code>textContent</code> can help prevent jiggle in
horizontally-arranged tabs
</strong>
. The size of the tab will accommodate this string rendered in
bold text.
</p>
<p>
Bolding is used in our design patterns to indicate a selected tab.
Without any intervention, textual tabs will jiggle around when
they are selected. This has a simple cause: bold text takes up
more room.
</p>
<Library.Demo
title="Tabs without textContent (jiggle when selected)"
withSource
>
<TabList classes="gap-x-6">
<Tab
selected={sidebarPanel === 'annotations'}
onClick={() => setSidebarPanel('annotations')}
>
Annotations
<span className="relative bottom-[3px] left-[2px] text-[10px]">
{52}
</span>
</Tab>
<Tab
selected={sidebarPanel === 'pageNotes'}
onClick={() => setSidebarPanel('pageNotes')}
>
Page Notes
<span className="relative bottom-[3px] left-[2px] text-[10px]">
{48}
</span>
</Tab>
<Tab
selected={sidebarPanel === 'orphans'}
onClick={() => setSidebarPanel('orphans')}
>
Orphans
<span className="relative bottom-[3px] left-[2px] text-[10px]">
{4}
</span>
</Tab>
</TabList>
</Library.Demo>
<p>
For tabs that have a simple text label, setting{' '}
<code>textContent</code> to that string avoids the jiggle. Text
will still change size (bold text is larger) but the tabs
themselves do not move.
</p>
<Library.Demo title="Tabs with textContent (no jiggle)" withSource>
<TabList classes="gap-x-6">
<Tab
selected={sidebarPanel2 === 'annotations'}
onClick={() => setSidebarPanel2('annotations')}
textContent="Annotations"
>
Annotations
</Tab>
<Tab
selected={sidebarPanel2 === 'pageNotes'}
onClick={() => setSidebarPanel2('pageNotes')}
textContent="Page Notes"
>
Page Notes
</Tab>
<Tab
selected={sidebarPanel2 === 'orphans'}
onClick={() => setSidebarPanel2('orphans')}
textContent="Orphans"
>
Orphans
</Tab>
</TabList>
</Library.Demo>
<p>
For tabs with styled or dynamic content, <code>textContent</code>{' '}
can be set to an estimated {'"widest-possible-text-content"'}{' '}
value. A trial and error approach worked here:
</p>
<Library.Demo
title="Tabs with estimated widest-value textContent"
withSource
>
<TabList classes="gap-x-6">
<Tab
selected={sidebarPanel3 === 'annotations'}
onClick={() => setSidebarPanel3('annotations')}
textContent="Annotations##"
>
Annotations
<span className="relative bottom-[3px] left-[2px] text-[10px]">
{52}
</span>
</Tab>
<Tab
selected={sidebarPanel3 === 'pageNotes'}
onClick={() => setSidebarPanel3('pageNotes')}
textContent="Page Notes##"
>
Page Notes
<span className="relative bottom-[3px] left-[2px] text-[10px]">
{56}
</span>
</Tab>
<Tab
selected={sidebarPanel3 === 'orphans'}
onClick={() => setSidebarPanel3('orphans')}
textContent="Orphans##"
>
Orphans
<span className="relative bottom-[3px] left-[2px] text-[10px]">
{2}
</span>
</Tab>
</TabList>
</Library.Demo>
</Library.Example>
</Library.Pattern>
</Library.Section>
<Library.Section
title="TabList"
intro={
<p>
<code>TabList</code> is a presentational component that provides a{' '}
<code>{'role="tablist"'}</code> container and arrow-key navigation
as outlined by{' '}
<a href="https://www.w3.org/WAI/ARIA/apg/patterns/tabpanel/">
WAI-ARIA authoring practices
</a>
.
</p>
}
>
<Library.Pattern title="Status">
<p>
<code>TabList</code> is a new component.
</p>
</Library.Pattern>
<Library.Pattern title="Usage">
<Next.Usage componentName="TabList" />
<Library.Example>
<p>
This example demonstrates a full Tab pattern with{' '}
<code>TabList</code>, <code>Tab</code> and some tabpanels. The
tabpanels have been made focusable here as they contain no
focusable elements: pressing <kbd>tab</kbd> when in the tablist
will move focus to the active tabpanel.
</p>
<Library.Demo withSource title="Full Tab pattern example">
<div>
<TabList classes="w-[400px] gap-x-6 my-4">
<Tab
aria-controls="profile-panel"
textContent="Profile"
selected={prefPanel === 'profile'}
onClick={() => setPrefPanel('profile')}
>
Profile
</Tab>
<Tab
aria-controls="notifications-panel"
textContent="Notifications"
selected={prefPanel === 'notifications'}
onClick={() => setPrefPanel('notifications')}
>
Notifications
</Tab>
<Tab
aria-controls="account-panel"
textContent="Account"
selected={prefPanel === 'account'}
onClick={() => setPrefPanel('account')}
>
Account
</Tab>
</TabList>
<Card
classes={classnames(
{ hidden: prefPanel !== 'profile' },
'p-2 focus-visible-ring'
)}
id="profile-panel"
role="tabpanel"
tabIndex={0}
variant="flat"
>
Profile
</Card>
<Card
classes={classnames(
{ hidden: prefPanel !== 'notifications' },
'p-2 focus-visible-ring'
)}
id="notifications-panel"
role="tabpanel"
tabIndex={0}
variant="flat"
>
Notifications
</Card>
<Card
classes={classnames(
{ hidden: prefPanel !== 'account' },
'p-2 focus-visible-ring'
)}
id="account-panel"
role="tabpanel"
tabIndex={0}
variant="flat"
>
Account
</Card>
</div>
</Library.Demo>
</Library.Example>
</Library.Pattern>
</Library.Section>
</Library.Page>
);
}
7 changes: 7 additions & 0 deletions src/pattern-library/routes.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ import OverlayPage from './components/patterns/layout/OverlayPage';
import PointerButtonPage from './components/patterns/navigation/PointerButtonPage';
import LinkPage from './components/patterns/navigation/LinkPage';
import LinkButtonPage from './components/patterns/navigation/LinkButtonPage';
import TabPage from './components/patterns/navigation/TabPage';

// Legacy pattern-library pages

Expand Down Expand Up @@ -211,6 +212,12 @@ const routes: PlaygroundRoute[] = [
component: PointerButtonPage,
route: '/navigation-pointerbutton',
},
{
title: 'Tabs',
group: 'navigation',
component: TabPage,
route: '/navigation-tab',
},
{
route: '/components-buttons',
title: 'Buttons',
Expand Down

0 comments on commit e647f6f

Please sign in to comment.