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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vasike created an issue. See original summary.

vasike’s picture

Title: Add plural labels to Entity Types in Aggregator module » Add plural labels to Entity Types in Responsive image module
vasike’s picture

and here is the patch for this.

bojanz’s picture

Status: Needs review » Needs work
+ *     plural = "@count responsive image styles"

Needs a trailing comma.

vasike’s picture

Status: Needs work » Needs review
FileSize
853 bytes

indeed. patch updated
@ bojanz : thanks & sorry

bojanz’s picture

Status: Needs review » Reviewed & tested by the community

Thanks!

xjm’s picture

Status: Reviewed & tested by the community » Closed (duplicate)
Related issues: +#2702683: Add plural labels to entity types

Hi 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!

rosinegrean’s picture

Issue tags: -DCTransylvania