Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
As mentioned in #3027808: Improve labels shorten the status label to just Status: Published
. And as @berdir suggested I created this as a separate issue to commit this small change faster and leave the discussion about the author and date to the other issue.
Comment | File | Size | Author |
---|---|---|---|
#4 | interdiff-3030471-2-4.txt | 738 bytes | arpad.rozsa |
#4 | shorten-the-status-label-3030471-4.patch | 594 bytes | arpad.rozsa |
| |||
#2 | shorten-the-status-label-3030471-2.patch | 751 bytes | arpad.rozsa |
|
Comments
Comment #2
arpad.rozsa CreditAttribution: arpad.rozsa at MD Systems GmbH commentedComment #3
Berdirlooks like this was reformatted. Can we try to only change what is really necessary?
Note that translating things like that doesn't work. E.g. arabic is a left-to-right language (russian too I think), translatable things must always be combined, so lets use {% trans %} like node.html.twig and make the state a placeholder. That's still necessary when it's just State: @state, due to LTR languages.
Also interesting that this doesn't cause any test fails :)
Comment #4
arpad.rozsa CreditAttribution: arpad.rozsa at MD Systems GmbH commentedMade the changes only where it was necessary and used {% trans %} like you suggested.
Comment #5
BerdirThanks, not sure if it should be State or Status. https://english.stackexchange.com/questions/12958/status-vs-state, I'll never fully grasp that. State is closer to what it was before.
Comment #6
BerdirComment #8
samuel.mortensonLooks good to me. @Berdir typically "Status" would be something different than the state label, for example "Status: In Draft" feels better to me than "Status: Draft", but we don't have that level of detail coming from Workflows. Committed as-is.