Adminhtml: improved info for order sent status #4623
Conversation
github-actions
bot
added
Template : admin
|
This PR needs changes before merging. |
|
For change wordings please make suggestions via review (select lines & ctrl+h). |
| @@ -52,6 +47,10 @@ $orderStoreDate = $this->formatDate($_order->getCreatedAtStoreDate(), 'medium', | |||
| <td class="label"><label><?php echo Mage::helper('sales')->__('Order Status') ?></label></td> | |||
| <td class="value"><strong><span id="order_status"><?php echo $_order->getStatusLabel() ?></span></strong></td> | |||
| </tr> | |||
| <tr> | |||
| <td class="label"><label><?php echo Mage::helper('sales')->__('Order Confirmation Email') ?></label></td> | |||
There was a problem hiding this comment.
I would prefer a two-word label. These two variants are more appropriate.
Variant 1
| <td class="label"><label><?php echo Mage::helper('sales')->__('Order Confirmation Email') ?></label></td> | |
| <td class="label"><label><?php echo Mage::helper('sales')->__('Email Confirmation') ?></label></td> |
Variant 2
| <td class="label"><label><?php echo Mage::helper('sales')->__('Order Confirmation Email') ?></label></td> | |
| <td class="label"><label><?php echo Mage::helper('sales')->__('Order Confirmation') ?></label></td> |
I asked Copilot and Chatgpt for their opinions, both argued that "Order Confirmation" is more appropriate, as it covers confirmations sent by email, SMS, phone. Personally, I would prefer Email Confirmation because it specifies the confirmation method available in OpenMage.
If we are analyzing the label names in that block that obviously refer to the order, then the word "Order" should be removed from the "Order Date" and "Order Status" forms. Here is what it would look like in my opinion
There was a problem hiding this comment.
I disagree on that.
- "Email" suffix ... its no SMS, fax, ...
- "Order" prefix ... this block is displayed for orders, invoices, shippings ... so it could be missleading w/o it.
I really thought about how to name it - and it was also named "order confirmation email" before.
|
Regarding the positioning of the copy icon, the issue is still visible in the legacy theme.
|
|
Description (*)
See #4620 (comment)
Fixed Issues (if relevant)