{"action":"closed","issue":{"url":"https://api.github.com/repos/mui-org/material-ui/issues/10458","repository_url":"https://api.github.com/repos/mui-org/material-ui","labels_url":"https://api.github.com/repos/mui-org/material-ui/issues/10458/labels{/name}","comments_url":"https://api.github.com/repos/mui-org/material-ui/issues/10458/comments","events_url":"https://api.github.com/repos/mui-org/material-ui/issues/10458/events","html_url":"https://github.com/mui-org/material-ui/issues/10458","id":300505541,"number":10458,"title":"Accessibility Issue: Dialogs should trap focus","user":{"login":"harinair","id":763819,"avatar_url":"https://avatars0.githubusercontent.com/u/763819?v=4","gravatar_id":"","url":"https://api.github.com/users/harinair","html_url":"https://github.com/harinair","followers_url":"https://api.github.com/users/harinair/followers","following_url":"https://api.github.com/users/harinair/following{/other_user}","gists_url":"https://api.github.com/users/harinair/gists{/gist_id}","starred_url":"https://api.github.com/users/harinair/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/harinair/subscriptions","organizations_url":"https://api.github.com/users/harinair/orgs","repos_url":"https://api.github.com/users/harinair/repos","events_url":"https://api.github.com/users/harinair/events{/privacy}","received_events_url":"https://api.github.com/users/harinair/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-02-27T06:11:57Z","updated_at":"2018-02-27T08:03:32Z","closed_at":"2018-02-27T08:03:31Z","author_association":"NONE","body":"When a dialog is drawn with multiple elements, keyboard navigation allows the user to switch focus. But when the last element is focused and the tab key is hit, the focus should switch back to the first element. This is reported as fixed in #4384 but still repro'd in 0.20.\r\n\r\n- [x] I have searched the [issues](https://github.com/mui-org/material-ui/issues) of this repository and believe that this is not a duplicate.\r\n\r\n## Steps to Reproduce (for bugs)\r\n\r\nCreate a dialog with a few fields. Use tab to switch focus to the last element. Hit tab once again. Check which element has the focus. Accessible dialogs should switch focus to the first element inside the dialog.\r\n\r\n## Context\r\nThis is an accessibility issue. Reproducible in Material UI demo page. Here is how Material UI behaves:\r\nhttp://www.material-ui.com/#/components/dialog\r\nOpen a dialog and try tabbing. It tabs through all the fields in the page.\r\n\r\nThis is the expected behavior from a dialog that is accessible:\r\nhttps://material.angular.io/components/dialog/overview\r\nOpen a dialog and try tabbing. It tabs through *only* the fields in the *dialog*.\r\n\r\n\r\n| Tech         | Version |\r\n|--------------|---------|\r\n| Material-UI  |   0.20.0   |\r\n\r\n"}}
{"actor":{"display_login":"oliviertassinari"}}