From 4e4058bfa470be10a56064a7157e1c8a7887f603 Mon Sep 17 00:00:00 2001
From: cienislaw <cienislaw@post.pl>
Date: Sat, 12 Apr 2025 11:40:33 +0200
Subject: [PATCH 1/2] Add carrier table labels

Most fixes in Niara comes to non-advanced payment api 5-step checkout version. This is a first step to fix this issue and mimic those changes.
---
 order-carrier-advanced.tpl     | 14 ++++++++++---
 order-carrier-opc-advanced.tpl | 38 ++++++++++++++++++++--------------
 2 files changed, 34 insertions(+), 18 deletions(-)

diff --git a/order-carrier-advanced.tpl b/order-carrier-advanced.tpl
index 0557904..a9063e4 100644
--- a/order-carrier-advanced.tpl
+++ b/order-carrier-advanced.tpl
@@ -41,9 +41,12 @@
                     <table class="resume table table-bordered{if !$option.unique_carrier} hide{/if}">
                       <tr>
                         <td class="delivery_option_radio">
-                          <input id="delivery_option_{$id_address|intval}_{$option@index}" class="delivery_option_radio" type="radio" name="delivery_option[{$id_address|intval}]" data-key="{$key}" data-id_address="{$id_address|intval}" value="{$key}"{if isset($delivery_option[$id_address]) && $delivery_option[$id_address] == $key} checked="checked"{/if}><span class="label-text"></span>
+                          <label for="delivery_option_{$id_address|intval}_{$option@index}">
+                            <input id="delivery_option_{$id_address|intval}_{$option@index}" class="delivery_option_radio" type="radio" name="delivery_option[{$id_address|intval}]" data-key="{$key}" data-id_address="{$id_address|intval}" value="{$key}"{if isset($delivery_option[$id_address]) && $delivery_option[$id_address] == $key} checked="checked"{/if}><span class="label-text"></span>
+                          </label>
                         </td>
                         <td class="delivery_option_logo">
+                          <label for="delivery_option_{$id_address|intval}_{$option@index}">
                           {foreach $option.carrier_list as $carrier}
                             {if $carrier.logo}
                               <img class="order_carrier_logo" src="{$carrier.logo|escape:'htmlall':'UTF-8'}" alt="{$carrier.instance->name|escape:'htmlall':'UTF-8'}">
@@ -52,8 +55,10 @@
                               {if !$carrier@last} - {/if}
                             {/if}
                           {/foreach}
+                          </label>
                         </td>
                         <td>
+                          <label for="delivery_option_{$id_address|intval}_{$option@index}">
                           {if $option.unique_carrier}
                             {foreach $option.carrier_list as $carrier}
                               <strong>{$carrier.instance->name|escape:'htmlall':'UTF-8'}</strong>
@@ -74,9 +79,11 @@
                               <span class="best_grade best_grade_price">{l s='The best price'}</span>
                             {/if}
                           {/if}
+                          </label>
                         </td>
                         <td class="delivery_option_price">
-                          <div class="delivery_option_price">
+                          <label for="delivery_option_{$id_address|intval}_{$option@index}">
+                            <div>
                             {if $option.total_price_with_tax && !$option.is_free && (!isset($free_shipping) || (isset($free_shipping) && !$free_shipping))}
                               {if $use_taxes == 1}
                                 {if $priceDisplay == 1}
@@ -90,7 +97,8 @@
                             {else}
                               {l s='Free'}
                             {/if}
-                          </div>
+                            </div>
+                          </label>
                         </td>
                       </tr>
                     </table>
diff --git a/order-carrier-opc-advanced.tpl b/order-carrier-opc-advanced.tpl
index db8122d..dfcacbe 100644
--- a/order-carrier-opc-advanced.tpl
+++ b/order-carrier-opc-advanced.tpl
@@ -27,9 +27,12 @@
                     <table class="resume table table-bordered{if !$option.unique_carrier} hide{/if}">
                       <tr>
                         <td class="delivery_option_radio">
-                          <input id="delivery_option_{$id_address|intval}_{$option@index}" class="delivery_option_radio" type="radio" name="delivery_option[{$id_address|intval}]" data-key="{$key}" data-id_address="{$id_address|intval}" value="{$key}"{if isset($delivery_option[$id_address]) && $delivery_option[$id_address] == $key} checked="checked"{/if}><span class="label-text"></span>
+                          <label for="delivery_option_{$id_address|intval}_{$option@index}">
+                            <input id="delivery_option_{$id_address|intval}_{$option@index}" class="delivery_option_radio" type="radio" name="delivery_option[{$id_address|intval}]" data-key="{$key}" data-id_address="{$id_address|intval}" value="{$key}"{if isset($delivery_option[$id_address]) && $delivery_option[$id_address] == $key} checked="checked"{/if}><span class="label-text"></span>
+                          </label>
                         </td>
                         <td class="delivery_option_logo">
+                          <label for="delivery_option_{$id_address|intval}_{$option@index}">
                           {foreach $option.carrier_list as $carrier}
                             {if $carrier.logo}
                               <img class="order_carrier_logo" src="{$carrier.logo|escape:'htmlall':'UTF-8'}" alt="{$carrier.instance->name|escape:'htmlall':'UTF-8'}">
@@ -38,15 +41,17 @@
                               {if !$carrier@last} - {/if}
                             {/if}
                           {/foreach}
+                          </label>
                         </td>
                         <td>
+                          <label for="delivery_option_{$id_address|intval}_{$option@index}">
                           {if $option.unique_carrier}
                             {foreach $option.carrier_list as $carrier}
                               <strong>{$carrier.instance->name|escape:'htmlall':'UTF-8'}</strong>
+                              {if isset($carrier.instance->delay[$cookie->id_lang])}
+                                <br>{l s='Delivery time:'}&nbsp;{$carrier.instance->delay[$cookie->id_lang]|escape:'htmlall':'UTF-8'}
+                              {/if}
                             {/foreach}
-                            {if isset($carrier.instance->delay[$cookie->id_lang])}
-                              <br>{l s='Delivery time:'}&nbsp;{$carrier.instance->delay[$cookie->id_lang]|escape:'htmlall':'UTF-8'}
-                            {/if}
                           {/if}
                           {if count($option_list) > 1}
                             <br>
@@ -60,9 +65,11 @@
                               <span class="best_grade best_grade_price">{l s='The best price'}</span>
                             {/if}
                           {/if}
+                          </label>
                         </td>
                         <td class="delivery_option_price">
-                          <div class="delivery_option_price">
+                          <label for="delivery_option_{$id_address|intval}_{$option@index}">
+                            <div>
                             {if $option.total_price_with_tax && !$option.is_free && (!isset($free_shipping) || (isset($free_shipping) && !$free_shipping))}
                               {if $use_taxes == 1}
                                 {if $priceDisplay == 1}
@@ -76,7 +83,8 @@
                             {else}
                               {l s='Free'}
                             {/if}
-                          </div>
+                            </div>
+                          </label>
                         </td>
                       </tr>
                     </table>
@@ -261,9 +269,9 @@
         <p class="carrier_title">{l s='Leave a message'}</p>
         <div>
           <p>{l s='If you would like to add a comment about your order, please write it in the field below.'}</p>
-        <textarea class="form-control" cols="120" rows="2" name="message" id="message">{strip}
-            {if isset($oldMessage)}{$oldMessage|escape:'html':'UTF-8'}{/if}
-        {/strip}</textarea>
+            <textarea class="form-control" cols="120" rows="2" name="message" id="message">{strip}
+                {if isset($oldMessage)}{$oldMessage|escape:'html':'UTF-8'}{/if}
+            {/strip}</textarea>
         </div>
       {/if}
       {if $recyclablePackAllowed}
@@ -287,12 +295,12 @@
             {if $gift_wrapping_price > 0}
               &nbsp;<i>({l s='Additional cost of'}
               <span class="price" id="gift-price">
-                    {if $priceDisplay == 1}
-                      {convertPrice price=$total_wrapping_tax_exc_cost}
-                    {else}
-                      {convertPrice price=$total_wrapping_cost}
-                    {/if}
-                </span>
+                {if $priceDisplay == 1}
+                  {convertPrice price=$total_wrapping_tax_exc_cost}
+                {else}
+                  {convertPrice price=$total_wrapping_cost}
+                {/if}
+              </span>
               {if $use_taxes && $display_tax_label}
                 {if $priceDisplay == 1}
                   {l s='(tax excl.)'}

From 6691b0a03b710ce1b5266c30dfb18da0980f2307 Mon Sep 17 00:00:00 2001
From: cienislaw <cienislaw@post.pl>
Date: Sat, 12 Apr 2025 11:44:12 +0200
Subject: [PATCH 2/2] Remove extra child class

Parent TD has already delivery_option_price which sets width of that cell. The same class set on child DIV shrinks delivery price text even more.
---
 order-carrier.tpl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/order-carrier.tpl b/order-carrier.tpl
index fd058b9..b95e6e8 100644
--- a/order-carrier.tpl
+++ b/order-carrier.tpl
@@ -85,7 +85,7 @@
                         </td>
                         <td class="delivery_option_price">
                           <label for="delivery_option_{$id_address|intval}_{$option@index}">
-                            <div class="delivery_option_price">
+                            <div>
                             {if $option.total_price_with_tax && !$option.is_free && (!isset($free_shipping) || (isset($free_shipping) && !$free_shipping))}
                               {if $use_taxes == 1}
                                 {if $priceDisplay == 1}