Suppressing user avatar link when no avatar file specified, and addin…

…g an alt text for when one does" (#6936)
samvera · Nov 1, 2024 · 624a2c9 · 624a2c9
1 parent 542b449
commit 624a2c9
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/app/views/hyrax/users/_user_row.html.erb b/app/views/hyrax/users/_user_row.html.erb
@@ -1,8 +1,11 @@
 <tr>
-  <td><%= link_to hyrax.user_path(user) do %>
-        <%= image_tag(user.avatar.url(:thumb), width: 30) if user.avatar.file %>
+   <td>
+     <% if user.avatar.file %>
+      <%= link_to hyrax.user_path(user) do %>
+        <%= image_tag(user.avatar.url(:thumb), width: 30, alt: user.name) %>
       <% end %>
-  </td>
+     <% end %>
+   </td>
    <td><%= link_to user.name, hyrax.user_path(user) %></td>
    <td><%= link_to user.user_key, hyrax.user_path(user) %></td>
    <td><%= user.department %></td>