The current label for placed orders is just the order number. This is a change from Commerce 1.x, where we included the word "Order" in the label, and I couldn't find any closed issues discussion the change.

It creates an inconsistent UX through the interface where you'd expect to see "Order 123" atop a view page or in a breadcrumb, and it led us to put the word "order" into various messages where it didn't need to be. That wasn't a terrible issue until we fixed the lack of a label callback for draft orders / carts.

The proposal in this ticket is to update the order entity label to "Order [order:number]" as it was in Commerce 1.x and to review any necessary interface text changes throughout the codebase.

Comments

rszrama created an issue. See original summary.

jsacksick’s picture

Status: Active » Needs review
StatusFileSize
new3.75 KB
jsacksick’s picture

StatusFileSize
new7.5 KB
rszrama’s picture

-    return $this->t('Are you sure you want to unlock the order %label?', [
+    return $this->t('Are you sure you want to unlock the %label?', [

I think this should just be "unlock %label" in line with the others.

jsacksick’s picture

StatusFileSize
new7.49 KB
jsacksick’s picture

Status: Needs review » Fixed

Committed.

  • jsacksick committed 8a3c0c5 on 8.x-2.x
    Issue #3314990 by jsacksick, rszrama: Update the order entity label to "...

  • jsacksick committed 15f0ed8 on 3.0.x
    Issue #3314990 by jsacksick, rszrama: Update the order entity label to "...
fenstrat’s picture

This change was missed in a couple spots, filed #3316680: Update "Order [order:number]" messages in order module as the follow up.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.