From 81f1eb918bf09adeffcec0e99e6ba2bdb1f3629f Mon Sep 17 00:00:00 2001
From: Katsuya HIDAKA <hidakatsuya@gmail.com>
Date: Mon, 20 Jan 2025 15:17:42 +0900
Subject: Fix the member table layout breaks due to .icon class on td elements

---
 app/assets/stylesheets/application.css        | 2 +-
 app/views/projects/settings/_members.html.erb | 8 +++++---
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/app/assets/stylesheets/application.css b/app/assets/stylesheets/application.css
index e67a6e3b2..60f2c9a14 100644
--- a/app/assets/stylesheets/application.css
+++ b/app/assets/stylesheets/application.css
@@ -437,7 +437,7 @@ tr.version.closed, tr.version.closed a { color: #999; }
 tr.version:not(.shared) td.name { padding-left: 20px; }
 tr.version td.date, tr.version td.status, tr.version td.sharing { text-align: center; white-space:nowrap; }
 
-tr.member td.icon-user, #principals_for_new_member .icon-user {background:transparent;}
+tr.member .icon-user, #principals_for_new_member .icon-user {background:transparent;}
 #principals_for_new_member svg, #principals_for_new_member img {margin-right: 4px;}
 
 tr.user td {width:13%;white-space: nowrap;}
diff --git a/app/views/projects/settings/_members.html.erb b/app/views/projects/settings/_members.html.erb
index 6e13808b6..abe4ed996 100644
--- a/app/views/projects/settings/_members.html.erb
+++ b/app/views/projects/settings/_members.html.erb
@@ -19,9 +19,11 @@
   <% members.each do |member| %>
   <% next if member.new_record? %>
   <tr id="member-<%= member.id %>" class="member">
-  <td class="name icon icon-<%= member.principal.class.name.downcase %>">
-    <%= principal_icon(member.principal) %>
-    <%= link_to_user member.principal %>
+  <td class="name">
+    <span class="icon">
+      <%= principal_icon(member.principal) %>
+      <%= link_to_user member.principal %>
+    </span>
   </td>
   <td class="roles">
     <span id="member-<%= member.id %>-roles"><%= member.roles.sort.collect(&:to_s).join(', ') %></span>
-- 
2.44.0