Follow-up to #2702669: Add plural labels to Entity Types in Aggregator module
Problem/Motivation
As it has been introduced in #1850080: Entity type labels lack plurality, cannot generate UI text based on label if plural is needed now we have a way to specify plural label for any entity type. Let's do it for Responsive image module.
Proposed resolution
Add plural labels to all entity types defined by Responsive image module. More information can be found in the change record: https://www.drupal.org/node/2689949
Remaining tasks
Write the patch.
User interface changes
Nope.
API changes
Nope.
Data model changes
Nope.
Comment | File | Size | Author |
---|---|---|---|
#5 | responsive_image_entity_type_plural_labels-2702745-5.patch | 853 bytes | vasike |
#3 | responsive_image_entity_type_plural_labels-2702745-3.patch | 852 bytes | vasike |
Comments
Comment #2
vasikeComment #3
vasikeand here is the patch for this.
Comment #4
bojanz CreditAttribution: bojanz at Centarro commentedNeeds a trailing comma.
Comment #5
vasikeindeed. patch updated
@ bojanz : thanks & sorry
Comment #6
bojanz CreditAttribution: bojanz at Centarro commentedThanks!
Comment #7
xjmHi DCTransilvania contributors and others -- can you please comment on #2702683: Add plural labels to entity types to receive credit? Splitting up patches per-module is not ideal in most cases, especially for a single, simple change, so let's use a combined patch. Thank you!
Comment #8
rosinegrean CreditAttribution: rosinegrean at PitechPlus commented