Skip to content

Commit

Permalink
Gritter Accessibility Changes
Browse files Browse the repository at this point in the history
Adding in role to the main gritter item wrapper, as well as changing the
close div to a link tag to allow it to be readable, as well as able to
be closed via keyboard
  • Loading branch information
heitzke committed Jul 15, 2013
1 parent 5136539 commit 25bdf6f
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions css/jquery.gritter.css
Expand Up @@ -67,6 +67,7 @@
cursor:pointer;
width:30px;
height:30px;
text-indent:-9999em;
}
.gritter-title {
font-size:14px;
Expand Down
4 changes: 2 additions & 2 deletions js/jquery.gritter.js
Expand Up @@ -78,9 +78,9 @@
_custom_timer: 0,
_item_count: 0,
_is_setup: 0,
_tpl_close: '<div class="gritter-close"></div>',
_tpl_close: '<a class="gritter-close" href="javascript:void(0);" tabindex="1">Close Notification</a>',
_tpl_title: '<span class="gritter-title">[[title]]</span>',
_tpl_item: '<div id="gritter-item-[[number]]" class="gritter-item-wrapper [[item_class]]" style="display:none"><div class="gritter-top"></div><div class="gritter-item">[[close]][[image]]<div class="[[class_name]]">[[title]]<p>[[text]]</p></div><div style="clear:both"></div></div><div class="gritter-bottom"></div></div>',
_tpl_item: '<div id="gritter-item-[[number]]" class="gritter-item-wrapper [[item_class]]" style="display:none" role="alert"><div class="gritter-top"></div><div class="gritter-item">[[close]][[image]]<div class="[[class_name]]">[[title]]<p>[[text]]</p></div><div style="clear:both"></div></div><div class="gritter-bottom"></div></div>',
_tpl_wrap: '<div id="gritter-notice-wrapper"></div>',

/**
Expand Down
2 changes: 1 addition & 1 deletion js/jquery.gritter.min.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 25bdf6f

Please sign in to comment.