summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Gerhardt <code@dgerhardt.net>2015-07-25 14:14:51 +0200
committerDaniel Gerhardt <code@dgerhardt.net>2015-07-30 01:28:33 +0200
commit9a595b9af735d713e86720c048b5fadba1db0f63 (patch)
treecd20ad07e4a34251aec0c43609043459d53b9592
parentf1db513a69503e652ab26cc4f985d9c6588c65cb (diff)
downloadgitlab-ci-9a595b9af735d713e86720c048b5fadba1db0f63.tar.gz
Do not use image's file name as alternative text
`image_tag` by default uses the image's file name for the img tag's alt attribute. This is not preferable in many cases. E.g. it clutters the UI with a long hash string if a gravatar could not be loaded. To fix this, the `alt` option is set for `image_tag` calls.
-rw-r--r--app/helpers/application_helper.rb2
-rw-r--r--app/views/layouts/_nav.html.haml2
-rw-r--r--app/views/user_sessions/show.html.haml2
3 files changed, 3 insertions, 3 deletions
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 069005f..8eeb409 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -1,6 +1,6 @@
module ApplicationHelper
def loader_html
- image_tag 'loader.gif'
+ image_tag 'loader.gif', alt: 'Loading'
end
def gravatar_icon(user_email = '', size = nil)
diff --git a/app/views/layouts/_nav.html.haml b/app/views/layouts/_nav.html.haml
index ab2308a..d571716 100644
--- a/app/views/layouts/_nav.html.haml
+++ b/app/views/layouts/_nav.html.haml
@@ -21,7 +21,7 @@
%li
= link_to user_sessions_path do
.profile-holder
- = image_tag gravatar_icon(current_user.email, 64)
+ = image_tag gravatar_icon(current_user.email, 64), alt: ''
%span= current_user.name
%li
= link_to user_sessions_path, class: "logout", method: :delete do
diff --git a/app/views/user_sessions/show.html.haml b/app/views/user_sessions/show.html.haml
index 12aea2a..0faf2f7 100644
--- a/app/views/user_sessions/show.html.haml
+++ b/app/views/user_sessions/show.html.haml
@@ -1,4 +1,4 @@
-= image_tag gravatar_icon(current_user.email, 120), class: 'img-polaroid'
+= image_tag gravatar_icon(current_user.email, 120), class: 'img-polaroid', alt: ''
%h3
Hi, #{@user.name}