Skip to content

Commit

Permalink
Fix typos, improve titles and layout
Browse files Browse the repository at this point in the history
  • Loading branch information
dotherightthing committed Sep 24, 2020
1 parent 3be1186 commit 51bc553
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 4 deletions.
4 changes: 2 additions & 2 deletions cypress/integration/tabbed-carousel.js
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ describe('Tabbed Carousel', function () {
});

it('+ When the tab panel has the focus, moves focus to the first tab panel navigation button', function () {
cy.log('Nav moved after slide in order to have the active slide adjacent to the active tab in the tab sequence. This is in contrast with the WAI - ARIA Carousel example, which doesn\'t have tab navigation');
cy.log('Nav moved after slide in order to have the active slide adjacent to the active tab in the tab sequence. This is in contrast with the WAI-ARIA Carousel example, which doesn\'t have tab navigation');

cy.get('@testAnchor')
.tab().tab().tab();
Expand All @@ -207,7 +207,7 @@ describe('Tabbed Carousel', function () {
});

it('+ When the first tab panel navigation button has the focus, moves focus to the next tab panel navigation button', function () {
cy.log('Nav moved after slide in order to have the active slide adjacent to the active tab in the tab sequence. This is in contrast with the WAI - ARIA Carousel example, which doesn\'t have tab navigation');
cy.log('Nav moved after slide in order to have the active slide adjacent to the active tab in the tab sequence. This is in contrast with the WAI-ARIA Carousel example, which doesn\'t have tab navigation');

cy.get('@testAnchor')
.tab().tab().tab()
Expand Down
4 changes: 4 additions & 0 deletions scss/_shared.scss
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,10 @@ h2 {
margin: 0;
}

[id^='test-'] {
margin-bottom: 48px;
}

.test-focus-start {
background-color: $color-component-bg;
padding: 4px;
Expand Down
4 changes: 2 additions & 2 deletions tabbed-carousel.html
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<h1>Tabbed carousel</h1>
<p>See Cypress unit tests for specification.</p>
<div id="test-1">
<h2>Test 1</h2>
<h2>Example 1: Tabs with Manual Activation</h2>
<button class="test-focus-start" id="test-1-start">Start test</button>
<div class="tabbed-carousel" data-initial-selection="1">
<div role="tablist" aria-label="Choose photo to display">
Expand Down Expand Up @@ -65,7 +65,7 @@ <h2>Image 5</h2>
</div>
</div>
<div id="test-2">
<h2>Test 2: Selection follows focus</h2>
<h2>Example 2: Tabs with Automatic Activation</h2>
<button class="test-focus-start" id="test-2-start">Start test</button>
<div class="tabbed-carousel" data-selection-follows-focus="true" data-initial-selection="1">
<div role="tablist" aria-label="Choose photo to display">
Expand Down

0 comments on commit 51bc553

Please sign in to comment.