{"action":"created","issue":{"url":"https://api.github.com/repos/dequelabs/axe-core/issues/166","repository_url":"https://api.github.com/repos/dequelabs/axe-core","labels_url":"https://api.github.com/repos/dequelabs/axe-core/issues/166/labels{/name}","comments_url":"https://api.github.com/repos/dequelabs/axe-core/issues/166/comments","events_url":"https://api.github.com/repos/dequelabs/axe-core/issues/166/events","html_url":"https://github.com/dequelabs/axe-core/issues/166","id":133590725,"number":166,"title":"Performance improvements for utils.isHidden","user":{"login":"ckundo","id":108163,"avatar_url":"https://avatars.githubusercontent.com/u/108163?v=3","gravatar_id":"","url":"https://api.github.com/users/ckundo","html_url":"https://github.com/ckundo","followers_url":"https://api.github.com/users/ckundo/followers","following_url":"https://api.github.com/users/ckundo/following{/other_user}","gists_url":"https://api.github.com/users/ckundo/gists{/gist_id}","starred_url":"https://api.github.com/users/ckundo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ckundo/subscriptions","organizations_url":"https://api.github.com/users/ckundo/orgs","repos_url":"https://api.github.com/users/ckundo/repos","events_url":"https://api.github.com/users/ckundo/events{/privacy}","received_events_url":"https://api.github.com/users/ckundo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-02-14T23:09:04Z","updated_at":"2016-02-27T15:12:19Z","closed_at":null,"body":"I evaluated axe-core with Firefox performance tools, running it in http://www.html5accessibility.com/html5elements/ page. I saw that `utils.isHidden` costs about 20% of the total execution time of `axe.a11yCheck` call. On this particular page that equates to about 200ms. Thoughts on improving performance in that method? I'm happy to chip in or share ideas, but wanted to capture the issue first."},"comment":{"url":"https://api.github.com/repos/dequelabs/axe-core/issues/comments/189660060","html_url":"https://github.com/dequelabs/axe-core/issues/166#issuecomment-189660060","issue_url":"https://api.github.com/repos/dequelabs/axe-core/issues/166","id":189660060,"user":{"login":"dylanb","id":79245,"avatar_url":"https://avatars.githubusercontent.com/u/79245?v=3","gravatar_id":"","url":"https://api.github.com/users/dylanb","html_url":"https://github.com/dylanb","followers_url":"https://api.github.com/users/dylanb/followers","following_url":"https://api.github.com/users/dylanb/following{/other_user}","gists_url":"https://api.github.com/users/dylanb/gists{/gist_id}","starred_url":"https://api.github.com/users/dylanb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dylanb/subscriptions","organizations_url":"https://api.github.com/users/dylanb/orgs","repos_url":"https://api.github.com/users/dylanb/repos","events_url":"https://api.github.com/users/dylanb/events{/privacy}","received_events_url":"https://api.github.com/users/dylanb/received_events","type":"User","site_admin":false},"created_at":"2016-02-27T15:12:19Z","updated_at":"2016-02-27T15:12:19Z","body":"@jnurthen If you look at the code, you will see that it is a bit more complicated than just display:none https://github.com/dequelabs/axe-core/blob/master/lib/core/utils/is-hidden.js but we could possibly improve the performance somewhat by doing that test first and then the other tests. Anyone want to take a shot at doing that?"}}