From f4e477622b06030ac575cd1fd2bb5b21220c7405 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Oliver=20G=C3=BCnther?= <mail@oliverguenther.de>
Date: Sun, 19 Nov 2023 21:19:44 +0100
Subject: [PATCH 1/3] Make structured meeting the default

---
 .../meeting/app/views/meetings/_form.html.erb | 110 +++++++++---------
 .../spec/features/meetings_copy_spec.rb       |   2 +
 .../spec/features/meetings_new_spec.rb        |   7 ++
 3 files changed, 65 insertions(+), 54 deletions(-)

diff --git a/modules/meeting/app/views/meetings/_form.html.erb b/modules/meeting/app/views/meetings/_form.html.erb
index ff354c53445c..26cac65acb06 100644
--- a/modules/meeting/app/views/meetings/_form.html.erb
+++ b/modules/meeting/app/views/meetings/_form.html.erb
@@ -35,51 +35,53 @@ See COPYRIGHT and LICENSE files for more details.
     <%= f.text_field :title, :required => true, :size => 60, container_class: '-wide' %>
   </div>
 
-  <div class="form--field -required -align-start">
-    <label class="form--label" for="meeting_type"><%= Meeting.human_attribute_name(:type) %></label>
-    <div class="form--field-container">
-      <div class="op-tile-block">
-        <label class="op-tile-block--tile form--radio-button-container -wide"
-               for="meeting_type_classic"
-               data-test-selector="op-tile-block">
-          <div class="op-tile-block--content">
-            <%= styled_radio_button_tag 'meeting[type]',
-                                        'classic',
-                                        true,
-                                        no_label: true,
-                                        class: 'radio-button' %>
-            <div>
+  <% if @meeting.new_record? %>
+    <div class="form--field -required -align-start">
+      <label class="form--label" for="meeting_type"><%= Meeting.human_attribute_name(:type) %></label>
+      <div class="form--field-container">
+        <div class="op-tile-block">
+          <label class="op-tile-block--tile form--radio-button-container -wide"
+                 for="meeting_type_dynamic"
+                 data-test-selector="op-tile-block">
+            <div class="op-tile-block--content">
+              <%= styled_radio_button_tag 'meeting[type]',
+                                          'dynamic',
+                                          true,
+                                          no_label: true,
+                                          class: 'radio-button' %>
+              <div>
               <span data-test-selector="op-tile-block-title" class="op-tile-block--title">
-                <%= t('meeting.types.classic') %>
+                <%= t('meeting.types.structured') %>
               </span>
-              <p class="op-tile-block--description">
-                <%= t('meeting.types.classic_text') %>
-              </p>
+                <p class="op-tile-block--description">
+                  <%= t('meeting.types.structured_text') %>
+                </p>
+              </div>
             </div>
-          </div>
-        </label>
-        <label class="op-tile-block--tile form--radio-button-container -wide"
-               for="meeting_type_dynamic"
-               data-test-selector="op-tile-block">
-          <div class="op-tile-block--content">
-            <%= styled_radio_button_tag 'meeting[type]',
-                                        'dynamic',
-                                        false,
-                                        no_label: true,
-                                        class: 'radio-button' %>
-            <div>
+          </label>
+          <label class="op-tile-block--tile form--radio-button-container -wide"
+                 for="meeting_type_classic"
+                 data-test-selector="op-tile-block">
+            <div class="op-tile-block--content">
+              <%= styled_radio_button_tag 'meeting[type]',
+                                          'classic',
+                                          false,
+                                          no_label: true,
+                                          class: 'radio-button' %>
+              <div>
               <span data-test-selector="op-tile-block-title" class="op-tile-block--title">
-                <%= t('meeting.types.structured') %>
+                <%= t('meeting.types.classic') %>
               </span>
-              <p class="op-tile-block--description">
-                <%= t('meeting.types.structured_text') %>
-              </p>
+                <p class="op-tile-block--description">
+                  <%= t('meeting.types.classic_text') %>
+                </p>
+              </div>
             </div>
-          </div>
-        </label>
+          </label>
+        </div>
       </div>
     </div>
-  </div>
+  <% end %>
 
   <% if global_meeting_create_context? %>
     <div class="form--field -required">
@@ -110,7 +112,7 @@ See COPYRIGHT and LICENSE files for more details.
   <div class="form--field -required">
     <label for="meeting_start_date" class="form--label"><%= Meeting.human_attribute_name(:start_date) %>
       <span class="hidden-for-sighted">
-        <%= t(:label_start_date)%><%= t(:text_hint_date_format) %>
+        <%= t(:label_start_date) %><%= t(:text_hint_date_format) %>
       </span>
     </label>
 
@@ -128,13 +130,13 @@ See COPYRIGHT and LICENSE files for more details.
         </label>
       </label>
       <%= f.text_field :start_time_hour,
-            id: 'meeting-form-start-time',
-            required: true,
-            type:     'time',
-            no_label: true,
-            step:     5.minutes,
-            suffix:   Time.zone.to_s,
-            container_class: '-xslim' %>
+                       id: 'meeting-form-start-time',
+                       required: true,
+                       type: 'time',
+                       no_label: true,
+                       step: 5.minutes,
+                       suffix: Time.zone.to_s,
+                       container_class: '-xslim' %>
     </div>
   </div>
 
@@ -142,15 +144,15 @@ See COPYRIGHT and LICENSE files for more details.
     <label for="meeting-form-duration" class="form--label"><%= Meeting.human_attribute_name(:duration) %></label>
     <div class="form--field-container">
       <%= f.number_field :duration,
-            id: 'meeting-form-duration',
-            required: true,
-            size:     5,
-            no_label: true,
-            min:      0.00,
-            step:     0.25,
-            max:      168,
-            suffix:   t(:text_in_hours),
-            container_class: '-xslim' %>
+                         id: 'meeting-form-duration',
+                         required: true,
+                         size: 5,
+                         no_label: true,
+                         min: 0.00,
+                         step: 0.25,
+                         max: 168,
+                         suffix: t(:text_in_hours),
+                         container_class: '-xslim' %>
     </div>
   </div>
 
diff --git a/modules/meeting/spec/features/meetings_copy_spec.rb b/modules/meeting/spec/features/meetings_copy_spec.rb
index a7558b9ad4ee..0bd102932c95 100644
--- a/modules/meeting/spec/features/meetings_copy_spec.rb
+++ b/modules/meeting/spec/features/meetings_copy_spec.rb
@@ -94,6 +94,8 @@
     expect(page)
       .to have_field 'Time',       with: start_time.strftime("%H:%M")
 
+    choose 'Classic'
+
     click_button "Create"
 
     # Be on the new meeting's page with copied over attributes
diff --git a/modules/meeting/spec/features/meetings_new_spec.rb b/modules/meeting/spec/features/meetings_new_spec.rb
index 49ecffb621ca..d8885752d8da 100644
--- a/modules/meeting/spec/features/meetings_new_spec.rb
+++ b/modules/meeting/spec/features/meetings_new_spec.rb
@@ -87,6 +87,7 @@
           expect_angular_frontend_initialized # Wait for project dropdown to be ready
 
           new_page.set_title 'Some title'
+          new_page.set_type 'Classic'
           new_page.set_project project
 
           new_page.set_start_date '2013-03-28'
@@ -133,6 +134,7 @@
         before do
           new_page.visit!
           new_page.set_title 'Some title'
+          new_page.set_type 'Classic'
           new_page.set_start_date '2013-03-28'
           new_page.set_start_time '13:30'
           new_page.set_duration '1.5'
@@ -168,6 +170,7 @@
         expect_angular_frontend_initialized # Wait for project dropdown to be ready
 
         new_page.set_title 'Some title'
+        new_page.set_type 'Classic'
 
         new_page.set_project project
 
@@ -185,6 +188,7 @@
         before do
           new_page.visit!
           new_page.set_title 'Some title'
+          new_page.set_type 'Classic'
         end
 
         it 'renders a validation error' do
@@ -242,6 +246,7 @@
           new_page.visit!
 
           new_page.set_title 'Some title'
+          new_page.set_type 'Classic'
           new_page.set_start_date '2013-03-28'
           new_page.set_start_time '13:30'
           new_page.set_duration '1.5'
@@ -298,6 +303,7 @@
       it 'allows creating meeting in a project without members' do
         new_page.visit!
 
+        new_page.set_type 'Classic'
         new_page.set_title 'Some title'
 
         show_page = new_page.click_create
@@ -327,6 +333,7 @@
         new_page.visit!
 
         new_page.set_title 'Some title'
+        new_page.set_type 'Classic'
 
         show_page = new_page.click_create
 

From 129a44bf99021f41a01eaa537d09e5b278f92b33 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Oliver=20G=C3=BCnther?= <mail@oliverguenther.de>
Date: Sun, 19 Nov 2023 22:21:07 +0100
Subject: [PATCH 2/3] Set monday as start of week to fix team planner specs

---
 modules/calendar/spec/features/calendar_widget_spec.rb       | 2 +-
 .../spec/features/team_planner_context_menu_spec.rb          | 5 ++++-
 .../team_planner/spec/features/team_planner_dates_spec.rb    | 5 ++++-
 .../spec/features/team_planner_error_handling_spec.rb        | 4 +++-
 .../spec/features/team_planner_remove_event_spec.rb          | 5 ++++-
 modules/team_planner/spec/features/team_planner_spec.rb      | 5 ++++-
 .../features/team_planner_subproject_constraints_spec.rb     | 5 ++++-
 .../spec/features/team_planner_user_interaction_spec.rb      | 5 ++++-
 8 files changed, 28 insertions(+), 8 deletions(-)

diff --git a/modules/calendar/spec/features/calendar_widget_spec.rb b/modules/calendar/spec/features/calendar_widget_spec.rb
index 143cf914e96a..d7b3d2bd8fe3 100644
--- a/modules/calendar/spec/features/calendar_widget_spec.rb
+++ b/modules/calendar/spec/features/calendar_widget_spec.rb
@@ -30,7 +30,7 @@
 require_relative '../../../overviews/spec/support/pages/overview'
 require_relative '../support/pages/calendar'
 
-RSpec.describe 'Calendar Widget', :js, :with_cuprite do
+RSpec.describe 'Calendar Widget', :js, :with_cuprite, with_settings: { start_of_week: 1 } do
   let(:project) do
     create(:project, enabled_module_names: %w[work_package_tracking calendar_view])
   end
diff --git a/modules/team_planner/spec/features/team_planner_context_menu_spec.rb b/modules/team_planner/spec/features/team_planner_context_menu_spec.rb
index e91e1f99ced9..3054cd7ab240 100644
--- a/modules/team_planner/spec/features/team_planner_context_menu_spec.rb
+++ b/modules/team_planner/spec/features/team_planner_context_menu_spec.rb
@@ -2,7 +2,10 @@
 require_relative 'shared_context'
 require 'features/work_packages/table/context_menu/context_menu_shared_examples'
 
-RSpec.describe 'Work package table context menu', :js, with_ee: %i[team_planner_view] do
+RSpec.describe 'Work package table context menu',
+               :js,
+               with_ee: %i[team_planner_view],
+               with_settings: { start_of_week: 1 } do
   include_context 'with team planner full access'
 
   let!(:work_package) do
diff --git a/modules/team_planner/spec/features/team_planner_dates_spec.rb b/modules/team_planner/spec/features/team_planner_dates_spec.rb
index d14e2ca81ece..b940244ebc62 100644
--- a/modules/team_planner/spec/features/team_planner_dates_spec.rb
+++ b/modules/team_planner/spec/features/team_planner_dates_spec.rb
@@ -29,7 +29,10 @@
 require 'spec_helper'
 require_relative './shared_context'
 
-RSpec.describe 'Team planner working days', js: true, with_ee: %i[team_planner_view] do
+RSpec.describe 'Team planner working days',
+               js: true,
+               with_ee: %i[team_planner_view],
+               with_settings: { start_of_week: 1 } do
   include_context 'with team planner full access'
 
   context 'with week days defined' do
diff --git a/modules/team_planner/spec/features/team_planner_error_handling_spec.rb b/modules/team_planner/spec/features/team_planner_error_handling_spec.rb
index 172542249706..098d08b16312 100644
--- a/modules/team_planner/spec/features/team_planner_error_handling_spec.rb
+++ b/modules/team_planner/spec/features/team_planner_error_handling_spec.rb
@@ -29,7 +29,9 @@
 require 'spec_helper'
 require_relative './shared_context'
 
-RSpec.describe 'Team planner error handling', js: true do
+RSpec.describe 'Team planner error handling',
+               js: true,
+               with_settings: { start_of_week: 1 } do
   include_context 'with team planner full access'
 
   let!(:work_package) do
diff --git a/modules/team_planner/spec/features/team_planner_remove_event_spec.rb b/modules/team_planner/spec/features/team_planner_remove_event_spec.rb
index 19d6e5cc0c1b..48ba23a3dae9 100644
--- a/modules/team_planner/spec/features/team_planner_remove_event_spec.rb
+++ b/modules/team_planner/spec/features/team_planner_remove_event_spec.rb
@@ -30,7 +30,10 @@
 require_relative 'shared_context'
 require_relative '../support/components/add_existing_pane'
 
-RSpec.describe 'Team planner remove event', :js, with_ee: %i[team_planner_view] do
+RSpec.describe 'Team planner remove event',
+               :js,
+               with_ee: %i[team_planner_view],
+               with_settings: { start_of_week: 1 } do
   include_context 'with team planner full access'
 
   let!(:viewer_role) { create(:project_role, permissions: [:view_work_packages]) }
diff --git a/modules/team_planner/spec/features/team_planner_spec.rb b/modules/team_planner/spec/features/team_planner_spec.rb
index 36f34a104e77..31bf31b3a643 100644
--- a/modules/team_planner/spec/features/team_planner_spec.rb
+++ b/modules/team_planner/spec/features/team_planner_spec.rb
@@ -29,7 +29,10 @@
 require 'spec_helper'
 require_relative 'shared_context'
 
-RSpec.describe 'Team planner', :js, with_ee: %i[team_planner_view] do
+RSpec.describe 'Team planner',
+               :js,
+               with_ee: %i[team_planner_view],
+               with_settings: { start_of_week: 1 } do
   include_context 'with team planner full access'
 
   it 'hides the internally used filters' do
diff --git a/modules/team_planner/spec/features/team_planner_subproject_constraints_spec.rb b/modules/team_planner/spec/features/team_planner_subproject_constraints_spec.rb
index 53c9e9415791..70c4f32c53cb 100644
--- a/modules/team_planner/spec/features/team_planner_subproject_constraints_spec.rb
+++ b/modules/team_planner/spec/features/team_planner_subproject_constraints_spec.rb
@@ -29,7 +29,10 @@
 require 'spec_helper'
 require_relative 'shared_context'
 
-RSpec.describe 'Team planner constraints for a subproject', :js, with_ee: %i[team_planner_view] do
+RSpec.describe 'Team planner constraints for a subproject',
+               :js,
+               with_ee: %i[team_planner_view],
+               with_settings: { start_of_week: 1 } do
   include_context 'with team planner full access'
 
   let!(:other_user) do
diff --git a/modules/team_planner/spec/features/team_planner_user_interaction_spec.rb b/modules/team_planner/spec/features/team_planner_user_interaction_spec.rb
index 4a35fba33070..28782860bc35 100644
--- a/modules/team_planner/spec/features/team_planner_user_interaction_spec.rb
+++ b/modules/team_planner/spec/features/team_planner_user_interaction_spec.rb
@@ -29,7 +29,10 @@
 require 'spec_helper'
 require_relative 'shared_context'
 
-RSpec.describe 'Team planner drag&dop and resizing', :js, with_ee: %i[team_planner_view] do
+RSpec.describe 'Team planner drag&dop and resizing',
+               :js,
+               with_ee: %i[team_planner_view],
+               with_settings: { start_of_week: 1 } do
   include_context 'with team planner full access'
 
   let!(:other_user) do

From 8679a84690b5ff448e9e16935d85eddbaef4808b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Oliver=20G=C3=BCnther?= <mail@oliverguenther.de>
Date: Mon, 20 Nov 2023 08:32:20 +0100
Subject: [PATCH 3/3] Resize by more pixels

---
 modules/team_planner/spec/support/pages/team_planner.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/team_planner/spec/support/pages/team_planner.rb b/modules/team_planner/spec/support/pages/team_planner.rb
index 059df4be4550..42bc43b27cef 100644
--- a/modules/team_planner/spec/support/pages/team_planner.rb
+++ b/modules/team_planner/spec/support/pages/team_planner.rb
@@ -261,7 +261,7 @@ def change_wp_date_by_resizing(work_package, number_of_days:, is_start_date:)
 
       resizer = is_start_date ? wp_strip.find('.fc-event-resizer-start') : wp_strip.find('.fc-event-resizer-end')
 
-      drag_by_pixel(element: resizer, by_x: number_of_days * 180, by_y: 0) unless resizer.nil?
+      drag_by_pixel(element: resizer, by_x: number_of_days * 250, by_y: 0) unless resizer.nil?
     end
 
     def drag_wp_by_pixel(work_package, by_x, by_y)