Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Commit

Permalink
Accessibility improvement
Browse files Browse the repository at this point in the history
Because they don't have a valid href, <a> tags are neither keyboard focusable (tab after the textbox) nor presented as link. 
Since they have the class btn and look like buttons, I decided to apply the role button instead of link.
  • Loading branch information
FremyCompany authored Jun 16, 2016
1 parent a40c21e commit 2cd5180
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions Server/views/httpproxy.jade
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,10 @@ html(lang='en')
input#url(type='text', placeholder='enter website url')
div.button
div.spacebutton
a#inject.btn.btn-default.left Inspect with VORLON.JS
a#openSite.btn.btn-default.left Open website only
a#openDashboard.btn.btn-default.left Open dashboard only
a#error.message.left.error.hide URL is not valid
a#inject.btn.btn-default.left(role='button', tabindex='0') Inspect with VORLON.JS
a#openSite.btn.btn-default.left(role='button', tabindex='0') Open website only
a#openDashboard.btn.btn-default.left(role='button', tabindex='0') Open dashboard only
a#error.message.left.error.hide(role='button', tabindex='0')URL is not valid

script(type='text/javascript').
$(document).ready(function() {
Expand Down Expand Up @@ -109,4 +109,4 @@ html(lang='en')
urlinput.addEventListener("blur", function (e) {
checkUrl();
});
});
});

0 comments on commit 2cd5180

Please sign in to comment.