Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

Added an accessibility feature for tabs directive by switching tabs through keyboard #2304

Closed
wants to merge 4 commits into from
Closed

Added an accessibility feature for tabs directive by switching tabs through keyboard #2304

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
f502b8b
fix #2290 Added an accessibility feature for tab directive
justin-john Jun 7, 2014
d55e76d
Corrected code alignment in tabs spec.
justin-john Jun 7, 2014
464c73e
test(tabs): clean up trailing whitespaces
justin-john Jun 7, 2014
f8462ef
Merge branch 'master' of github.com:justin-john/bootstrap-1
justin-john Jun 7, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Corrected code alignment in tabs spec.
  • Loading branch information
@justin-john
justin-john committed Jun 7, 2014
commit d55e76d880c91992859eaebcf021bedb34f751b0
10 changes: 5 additions & 5 deletions src/tabs/test/tabs.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -795,14 +795,14 @@ describe('tabs', function() {
it('should switch tab on focus', function() {
titles().eq(0).find('a').click();

titles().eq(1).find('a').focus();
expect(titles().eq(1)).toHaveClass('active');
titles().eq(1).find('a').focus();
expect(titles().eq(1)).toHaveClass('active');
expect(contents().eq(1)).toHaveClass('active');
expect(titles().eq(0)).not.toHaveClass('active');
expect(titles().eq(0)).not.toHaveClass('active');
expect(contents().eq(0)).not.toHaveClass('active');

titles().eq(2).find('a').focus();
expect(titles().eq(2)).toHaveClass('active');
titles().eq(2).find('a').focus();
expect(titles().eq(2)).toHaveClass('active');
expect(contents().eq(2)).toHaveClass('active');
expect(titles().eq(1)).not.toHaveClass('active');
expect(contents().eq(1)).not.toHaveClass('active');
Expand Down