summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorClement Ho <ClemMakesApps@gmail.com>2017-04-12 19:55:14 -0500
committerClement Ho <ClemMakesApps@gmail.com>2017-04-12 19:55:14 -0500
commit9ec58d8a67c611a5544c0994a8ef9f350a94d2db (patch)
tree6c47d8cfd2560e9968679bee46d8e1b2b19327fa
parent18188a53466bf81adb583a4d5be7e21796b33ab3 (diff)
downloadgitlab-ce-9ec58d8a67c611a5544c0994a8ef9f350a94d2db.tar.gz
Add aria to icon
-rw-r--r--app/helpers/icons_helper.rb5
-rw-r--r--changelogs/unreleased/add-aria-to-icon.yml4
2 files changed, 9 insertions, 0 deletions
diff --git a/app/helpers/icons_helper.rb b/app/helpers/icons_helper.rb
index ab3ef454e1c..ef260cff182 100644
--- a/app/helpers/icons_helper.rb
+++ b/app/helpers/icons_helper.rb
@@ -7,6 +7,11 @@ module IconsHelper
# font-awesome-rails gem, but should we ever use a different icon pack in the
# future we won't have to change hundreds of method calls.
def icon(names, options = {})
+ if !options['aria-hidden'] and !options['aria-label']
+ # Add `aria-hidden` if there are no aria's set
+ options['aria-hidden'] = true
+ end
+
options.include?(:base) ? fa_stacked_icon(names, options) : fa_icon(names, options)
end
diff --git a/changelogs/unreleased/add-aria-to-icon.yml b/changelogs/unreleased/add-aria-to-icon.yml
new file mode 100644
index 00000000000..b1449bf16a6
--- /dev/null
+++ b/changelogs/unreleased/add-aria-to-icon.yml
@@ -0,0 +1,4 @@
+---
+title: Update icon() to automatically include aria-hidden if aria is not found
+merge_request:
+author: