Slack: Remove extra a11y attributes from close_flexpane button

jwyung · jwyung · commit 46428caa0e3d · 2017-02-14T14:48:12.000-08:00
Based on the `close_flexpane` button markup on Slack now, we have:

`&lt;button class="btn_basic close_flexpane" aria-label="Close Right Sidebar" title="Close Right Sidebar" type="button"&gt;`

- Since this element is a `&lt;button&gt;`, the `role` attribute is no longer necessary
- Since this element has an `aria-label` attribute, we no longer need to add one
diff --git a/SlackA11yFixes.user.js b/SlackA11yFixes.user.js
@@ -19,12 +19,6 @@ function initial() {
 	// Same for the unread messages status, which appears below in DOM order but earlier visually.
 	if (elem = document.querySelector("#messages_container"))
 		elem.setAttribute("aria-owns", "messages_unread_status threads_view_banner monkey_scroll_wrapper_for_msgs_scroller_div monkey_scroll_wrapper_for_threads_msgs_scroller_div");
-	// Make close link for about channel pane accessible.
-	for (elem of document.querySelectorAll(".close_flexpane")) {
-		elem.setAttribute("role", "button");
-		// The content is a private use Unicode character. Use the title as the name.
-		elem.setAttribute("aria-label", elem.getAttribute("title"));
-	}
 }
 
 // Make the starred status accessible.
