Skip to content

Commit

Permalink
allow tabbing between groups
Browse files Browse the repository at this point in the history
  • Loading branch information
notwaldorf committed May 18, 2015
1 parent 2059a34 commit 23b9262
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 15 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
bower_components
3 changes: 2 additions & 1 deletion bower.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,11 @@
"ignore": [],
"dependencies": {
"iron-selector": "PolymerElements/iron-selector#^0.9.0",
"iron-a11y-keys-behavior": "PolymerElements/iron-a11y-keys-behavior#^0.9.0",
"polymer": "Polymer/polymer#^0.9.0"
},
"devDependencies": {
"web-component-tester": "Polymer/web-component-tester#^2.2.3",
"web-component-tester": "*",
"iron-doc-viewer": "PolymerElements/iron-doc-viewer#^0.9.0",
"test-fixture": "PolymerElements/test-fixture#^0.9.0",
"paper-radio-button": "PolymerElements/paper-radio-button#^0.9.0",
Expand Down
30 changes: 16 additions & 14 deletions paper-radio-group.html
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
<link rel="import" href="../polymer/polymer.html">
<link rel="import" href="../iron-selector/iron-selector.html">
<link rel="import" href="../paper-radio-button/paper-radio-button.html">
<link rel="import" href="../iron-a11y-keys-behavior/iron-a11y-keys-behavior.html">

<!--
`paper-radio-group` allows user to select only one radio button from a set.
Expand Down Expand Up @@ -57,8 +58,13 @@
Polymer({
is: 'paper-radio-group',

behaviors: [
Polymer.IronA11yKeysBehavior
],

hostAttributes: {
role: 'radiogroup'
role: 'radiogroup',
tabindex: 0
},

properties: {
Expand Down Expand Up @@ -86,9 +92,9 @@
}
},

listeners: {
// TODO: this can be replaced by <iron-a11y-keys> when it's ready.
keydown: '_onKeyDown'
keyBindings: {
'left up': '_selectPrevious',
'right down': '_selectNext',
},

_selectedChanged: function() {
Expand All @@ -99,16 +105,12 @@
});
},

_onKeyDown: function(event) {
if (event.keyCode === 37 || event.keyCode == 38) {
// Up/Left key goes back.
this.selected = this._previousNode();
} else if (event.keyCode === 39 || event.keyCode == 40) {
// Down/Right key advances.
this.selected = this._nextNode();
} else {
event.preventDefault();
}
_selectNext: function() {
this.selected = this._nextNode();
},

_selectPrevious: function() {
this.selected = this._previousNode();
},

/**
Expand Down

0 comments on commit 23b9262

Please sign in to comment.