From de020b190f4fade17f9f6603b953c69374eb3d40 Mon Sep 17 00:00:00 2001
From: yamakeeeeeeeeen <yamakeeeeeeeeen@gmail.com>
Date: Wed, 8 May 2024 10:25:23 +0900
Subject: [PATCH 1/4] =?UTF-8?q?chore:=20plan=E7=94=BB=E9=9D=A2=E3=81=AE?=
 =?UTF-8?q?=E3=83=A2=E3=83=90=E3=82=A4=E3=83=AB=E5=AF=BE=E5=BF=9C=E9=80=94?=
 =?UTF-8?q?=E4=B8=AD?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/views/plans/_mobile_table_row.html.erb | 7 +++++++
 app/views/plans/show.html.erb              | 7 ++++++-
 2 files changed, 13 insertions(+), 1 deletion(-)
 create mode 100644 app/views/plans/_mobile_table_row.html.erb

diff --git a/app/views/plans/_mobile_table_row.html.erb b/app/views/plans/_mobile_table_row.html.erb
new file mode 100644
index 00000000..b43068a6
--- /dev/null
+++ b/app/views/plans/_mobile_table_row.html.erb
@@ -0,0 +1,7 @@
+<% @selected = @plan.plan_schedules.map(&:schedule).any? { row.schedules.include?(_1) } %>
+<p class="text-xl font-normal mr-4 shrink-0"><%= row.start_end %></p>
+<% track_list.each do |track| %>
+  <% if row.tracks[track] %>
+    <div class="mt-4 p-2"><%= render("schedules/card", schedule: row[:schedule], mode: :plan, inactive: false) %></div>
+  <% end %>
+<% end %>
diff --git a/app/views/plans/show.html.erb b/app/views/plans/show.html.erb
index b954c228..19e34dfe 100644
--- a/app/views/plans/show.html.erb
+++ b/app/views/plans/show.html.erb
@@ -109,7 +109,7 @@
         <div class="mt-4 overflow-y-auto">
           <% @plans_table.each do |k, v| %>
             <div id="schedule-<%= k %>" class="schedule-table hidden">
-              <table class="h-full w-full border-collapse border-spacing-0">
+              <table class="hidden sm:table h-full w-full border-collapse border-spacing-0">
                 <thead>
                   <th class="p-4 text-sm border border-solid border-[rgb(214,211,208)] min-w-[184px] text-left"><%= I18n.t('table.start_end') %></th>
                   <th class="p-4 text-sm border border-solid border-[rgb(214,211,208)] w-auto text-center"><%= I18n.t('table.track') %></th>
@@ -133,6 +133,11 @@
                   <% end %>
                 </tbody>
               </table>
+              <div class="sm:hidden" id="mobile-table-<%= k %>">
+                <% @plans_table[k].rows.each do |row| %>
+                  <%= render partial: 'mobile_table_row', locals: { row: row, plan: @plan, track_list: @plans_table[k].track_list } %>
+                <% end %>
+              </div>
             </div>
           <% end %>
         </div>

From 2b64f0b7a80ed05ad959022b98a707c28d827991 Mon Sep 17 00:00:00 2001
From: kinoppyd <ppyd@kinoppyd.dev>
Date: Fri, 10 May 2024 18:15:06 +0900
Subject: [PATCH 2/4] Fix mobile row, no partial simple 2 lines

---
 app/views/plans/_mobile_table_row.html.erb | 7 -------
 app/views/plans/show.html.erb              | 5 +++--
 2 files changed, 3 insertions(+), 9 deletions(-)
 delete mode 100644 app/views/plans/_mobile_table_row.html.erb

diff --git a/app/views/plans/_mobile_table_row.html.erb b/app/views/plans/_mobile_table_row.html.erb
deleted file mode 100644
index b43068a6..00000000
--- a/app/views/plans/_mobile_table_row.html.erb
+++ /dev/null
@@ -1,7 +0,0 @@
-<% @selected = @plan.plan_schedules.map(&:schedule).any? { row.schedules.include?(_1) } %>
-<p class="text-xl font-normal mr-4 shrink-0"><%= row.start_end %></p>
-<% track_list.each do |track| %>
-  <% if row.tracks[track] %>
-    <div class="mt-4 p-2"><%= render("schedules/card", schedule: row[:schedule], mode: :plan, inactive: false) %></div>
-  <% end %>
-<% end %>
diff --git a/app/views/plans/show.html.erb b/app/views/plans/show.html.erb
index 19e34dfe..8bdd600b 100644
--- a/app/views/plans/show.html.erb
+++ b/app/views/plans/show.html.erb
@@ -134,8 +134,9 @@
                 </tbody>
               </table>
               <div class="sm:hidden" id="mobile-table-<%= k %>">
-                <% @plans_table[k].rows.each do |row| %>
-                  <%= render partial: 'mobile_table_row', locals: { row: row, plan: @plan, track_list: @plans_table[k].track_list } %>
+                <% v.each do |row| %>
+                  <p class="text-xl font-normal mr-4 shrink-0"><%= row[:time][:range] %></p>
+                  <div class="mt-4 p-2"><%= render("schedules/card", schedule: row[:schedule], mode: :plan, inactive: false) %></div>
                 <% end %>
               </div>
             </div>

From 09f3bb5093ecbfeb389b8b40be02a1ca424aa751 Mon Sep 17 00:00:00 2001
From: kinoppyd <ppyd@kinoppyd.dev>
Date: Fri, 10 May 2024 19:07:13 +0900
Subject: [PATCH 3/4] Add uniq ids for dialog (and other non-uniq) elements.

---
 app/views/schedules/_card.html.erb | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/app/views/schedules/_card.html.erb b/app/views/schedules/_card.html.erb
index b1d5df9b..2e4ae4cb 100644
--- a/app/views/schedules/_card.html.erb
+++ b/app/views/schedules/_card.html.erb
@@ -50,7 +50,8 @@
 
       <div class="flex justify-end flex-wrap-reverse gap-2">
         <% if mode == :plan && @plan.schedules.include?(schedule) %>
-          <div class="mr-auto" data-controller="dialog" data-dialog-element-id-value="edit-<%= schedule.id %>-memo-dialog">
+          <% memo_dialog_element_id = SecureRandom.uuid %>
+          <div class="mr-auto" data-controller="dialog" data-dialog-element-id-value="<%= memo_dialog_element_id %>">
             <button
               class="p-2 text-sm m-h-[calc(0.857143rem+18px)] normal-button"
               data-action="click->dialog#open"
@@ -58,7 +59,7 @@
               <svg stroke="currentColor" fill="currentColor" stroke-width="0" viewBox="0 0 512 512" class=" smarthr-ui-Icon" role="img" aria-hidden="true" focusable="false" height="12" width="12" xmlns="http://www.w3.org/2000/svg"><path d="M497.9 142.1l-46.1 46.1c-4.7 4.7-12.3 4.7-17 0l-111-111c-4.7-4.7-4.7-12.3 0-17l46.1-46.1c18.7-18.7 49.1-18.7 67.9 0l60.1 60.1c18.8 18.7 18.8 49.1 0 67.9zM284.2 99.8L21.6 362.4.4 483.9c-2.9 16.4 11.4 30.6 27.8 27.8l121.5-21.3 262.6-262.6c4.7-4.7 4.7-12.3 0-17l-111-111c-4.8-4.7-12.4-4.7-17.1 0zM124.1 339.9c-5.5-5.5-5.5-14.3 0-19.8l154-154c5.5-5.5 14.3-5.5 19.8 0s5.5 14.3 0 19.8l-154 154c-5.5 5.5-14.3 5.5-19.8 0zM88 424h48v36.3l-64.5 11.3-31.1-31.1L51.7 376H88v48z"></path></svg>
               <%= I18n.t('button.update_memo') %>
             </button>
-            <dialog id="edit-<%= schedule.id %>-memo-dialog" class="dialog max-w-[min(100vw-16px,800px)]">
+            <dialog id="<%= memo_dialog_element_id %>" class="dialog max-w-[min(100vw-16px,800px)]">
               <%= form_with(url: event_plan_path(@plan, event_name: @plan.event.name), method: :patch) do |f| %>
                 <div data-controller="word-counter" data-word-counter-max-value="<%= plan_memo_max_length %>" class="">
                   <div class="border-b-[1px] border-[rgb(214,211,208)] py-4 px-6 flex flex-col justify-start">
@@ -66,8 +67,8 @@
                   </div>
                   <div class="max-h-[calc(100vh-212px)] overflow-auto">
                     <div class="w-[656px] p-6">
-                      <%= f.hidden_field :edit_memo_schedule_id, value: schedule.id %>
-                      <%= f.text_area :memo, value: @plan.plan_schedules.find { _1.schedule == schedule }&.memo, class: "border opacity-100 rounded-md border border-[rgb(214,211,208)] bg-white p-2 text-[rgb(35,34,30)] w-full", data: { "word-counter-target": "source", action: "input->word-counter#calc" } %>
+                      <%= f.hidden_field :edit_memo_schedule_id, value: schedule.id, id: "#{schedule.id}-#{memo_dialog_element_id}" %>
+                      <%= f.text_area :memo, value: @plan.plan_schedules.find { _1.schedule == schedule }&.memo, id: SecureRandom.uuid, class: "border opacity-100 rounded-md border border-[rgb(214,211,208)] bg-white p-2 text-[rgb(35,34,30)] w-full", data: { "word-counter-target": "source", action: "input->word-counter#calc" } %>
                       <div class="font-xs mt-2 text-[rgb(112,109,101)]">
                         <span data-word-counter-target="counter"></span>/<%= plan_memo_max_length %>
                       </div>
@@ -89,8 +90,8 @@
           <div>
             <% if @plan.plan_schedules.map(&:schedule).include?(schedule) # to avoid n+1 %>
               <%= form_with(url: event_plan_url(@plan, event_name: @plan.event.name), method: 'PATCH' ) do |form| %>
-                <%= form.hidden_field :remove_schedule_id, value: schedule.id %>
-                <%= form.hidden_field :mode, value: mode %>
+                <%= form.hidden_field :remove_schedule_id, value: schedule.id, id: SecureRandom.uuid %>
+                <%= form.hidden_field :mode, value: mode, id: SecureRandom.uuid %>
                 <%= form.submit I18n.t('card.remove'), class: "remove-plan-button p-2 text-sm font-bold min-h-5 normal-button", data: { turbo_frame: "event_#{@event.id}" } %>
               <% end %>
             <% elsif @plan.new_record? %>

From 8b4ff44cb11b3946df8b128e7368b8e6323de007 Mon Sep 17 00:00:00 2001
From: yamakeeeeeeeeen <yamakeeeeeeeeen@gmail.com>
Date: Mon, 13 May 2024 10:53:58 +0900
Subject: [PATCH 4/4] =?UTF-8?q?fix:=20=E3=83=A2=E3=83=90=E3=82=A4=E3=83=AB?=
 =?UTF-8?q?=E3=81=AE=E6=99=82=E3=81=AB=E3=82=BF=E3=83=96=E3=81=8C=E5=B4=A9?=
 =?UTF-8?q?=E3=82=8C=E3=82=8B=E3=81=AE=E3=82=92=E4=BF=AE=E6=AD=A3?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/views/plans/show.html.erb      | 4 ++--
 app/views/schedules/index.html.erb | 4 ++--
 app/views/teams/show.html.erb      | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/views/plans/show.html.erb b/app/views/plans/show.html.erb
index 8bdd600b..ab456994 100644
--- a/app/views/plans/show.html.erb
+++ b/app/views/plans/show.html.erb
@@ -98,9 +98,9 @@
     <% else %>
       <div data-controller="schedule-table" class="max-w-[1120px] mx-auto">
         <div class="flex overflow-x-auto overflow-y-hidden gap-2 p-0">
-          <div class="relative flex sm:block justify-between flex-grow shrink-0 basis-auto m-1 before:absolute before:left-0 before:right-0 before:bottom-0 before:border-b-[1px]">
+          <div class="relative flex sm:block justify-center flex-grow shrink-0 basis-auto m-1 before:absolute before:left-0 before:right-0 before:bottom-0 before:border-b-[1px]">
             <% @plans_table.keys.map do |k| %>
-              <button data-action="click->schedule-table#switch" value="<%= k %>" class="tab-button bg-transparent appearance-none font-bold font-base text-[rgb(112,109,101)] h-10 solid border-0 border-b-[3px] px-2 sm:px-6 box-border border-b-transparent hover:bg-[rgb(248,247,246)] hover:text-[rgb(35,34,30)]">
+              <button data-action="click->schedule-table#switch" value="<%= k %>" class="tab-button bg-transparent appearance-none font-bold font-base text-[rgb(112,109,101)] h-10 solid border-0 border-b-[3px] px-4 sm:px-6 box-border border-b-transparent hover:bg-[rgb(248,247,246)] hover:text-[rgb(35,34,30)]">
                 <%= k %>
               </button>
             <% end %>
diff --git a/app/views/schedules/index.html.erb b/app/views/schedules/index.html.erb
index c52b5278..12fb4a43 100644
--- a/app/views/schedules/index.html.erb
+++ b/app/views/schedules/index.html.erb
@@ -21,9 +21,9 @@
     </div>
   </label>
   <div class="flex overflow-x-auto overflow-y-hidden gap-2 p-0">
-    <div class="relative flex sm:block justify-between flex-grow shrink-0 basis-auto m-1 before:absolute before:left-0 before:right-0 before:bottom-0 before:border-b-[1px]">
+    <div class="relative flex sm:block justify-center flex-grow shrink-0 basis-auto m-1 before:absolute before:left-0 before:right-0 before:bottom-0 before:border-b-[1px]">
       <% @schedule_table.days.each do |day| %>
-        <button data-action="click->schedule-table#switch" value="<%= day %>" class="tab-button bg-transparent appearance-none font-bold font-base text-[rgb(112,109,101)] h-10 solid border-0 border-b-[3px] px-2 sm:px-6 box-border border-b-transparent hover:bg-[rgb(248,247,246)] hover:text-[rgb(35,34,30)]">
+        <button data-action="click->schedule-table#switch" value="<%= day %>" class="tab-button bg-transparent appearance-none font-bold font-base text-[rgb(112,109,101)] h-10 solid border-0 border-b-[3px] px-4 sm:px-6 box-border border-b-transparent hover:bg-[rgb(248,247,246)] hover:text-[rgb(35,34,30)]">
           <%= day %>
         </button>
       <% end %>
diff --git a/app/views/teams/show.html.erb b/app/views/teams/show.html.erb
index 8421e536..570480c5 100644
--- a/app/views/teams/show.html.erb
+++ b/app/views/teams/show.html.erb
@@ -50,9 +50,9 @@
 
 <div data-controller="schedule-table" class="max-w-[1120px] mx-auto">
   <div class="flex overflow-x-auto overflow-y-hidden gap-2 p-0">
-    <div class="relative flex sm:block justify-between flex-grow shrink-0 basis-auto m-1 before:absolute before:left-0 before:right-0 before:bottom-0 before:border-b-[1px]">
+    <div class="relative flex sm:block justify-center flex-grow shrink-0 basis-auto m-1 before:absolute before:left-0 before:right-0 before:bottom-0 before:border-b-[1px]">
       <% @schedule_table.days.each do |day| %>
-        <button data-action="click->schedule-table#switch" value="<%= day %>" class="tab-button bg-transparent appearance-none font-bold font-base text-[rgb(112,109,101)] h-10 solid border-0 border-b-[3px] px-2 sm:px-6 box-border border-b-transparent hover:bg-[rgb(248,247,246)] hover:text-[rgb(35,34,30)]">
+        <button data-action="click->schedule-table#switch" value="<%= day %>" class="tab-button bg-transparent appearance-none font-bold font-base text-[rgb(112,109,101)] h-10 solid border-0 border-b-[3px] px-4 sm:px-6 box-border border-b-transparent hover:bg-[rgb(248,247,246)] hover:text-[rgb(35,34,30)]">
           <%= day %>
         </button>
       <% end %>