MediaElement.js: Missing plural forms for jump forward/back strings

[ideag]

Function wp_default_scripts() (https://core.trac.wordpress.org/browser/trunk/src/wp-includes/script-loader.php#L353 and https://core.trac.wordpress.org/browser/trunk/src/wp-includes/script-loader.php#L354) does not properly use _n() and related functions for plural forms of its strings. It just uses a single, plural string with a placeholder for a number, i.e. __( 'Jump forward %1 seconds' ).
While this works in English, it presents a problem when translating into languages that have more than one plural form, or more complicated rules for singular form usage. For example, in Lithuanian we also use singular form for 21, 1241 and any other number that ends with 1 (except for 11). And we have two plural forms - one for numbers ending with a zero, another for everything else. In current situation we can not have a proper translation here.

Depends on #20491 and related to #37287.

[ocean90]

Hello @ideag, those strings are the original strings which MediaElement.js (the library for the media player) is using:

• "Skip back %1 seconds": ​https://github.com/johndyer/mediaelement/blob/8adf73f1a2bec0b6867f83917ce8c491d01c1aa9/src/js/mep-feature-skipback.js#L7
• "Jump forward %1 seconds": ​https://github.com/johndyer/mediaelement/blob/8adf73f1a2bec0b6867f83917ce8c491d01c1aa9/src/js/mep-feature-jumpforward.js#L7

ME.js doesn't support plural forms, so this should probably ​reported upstream.

[ideag]

Hi @ocean90, I've ​reported this to ME.js, as per your suggestion.

[SergeyBiryukov]

Note that skipBackInterval and jumpForwardInterval are 30 by default, so unless someone changes the default values (which is possible, but unlikely), both strings would contain "30 seconds".

[ideag]

Replying to SergeyBiryukov:

Note that skipBackInterval and jumpForwardInterval are 30 by default, so unless someone changes the default values (which is possible, but unlikely), both strings would contain "30 seconds".

In that case, it would be helpful to provide these default values in translator comments.

[SergeyBiryukov]

In 38150:

I18N: Add a translator comment for two MediaElement.js strings added in [38089].

Props ideag.
See #37453, #37394.

[SergeyBiryukov]

In 43345:

I18N: Remove unused MediaElement.js strings.

Props metodiew.
See #42139. Fixes #37453.

[SergeyBiryukov]

In 47094:

I18N: Synchronize MediaElement.js translation strings with the currently bundled version.

Some strings were previously removed in [43345] and accidentally reintroduced in [44163].

Props ramiy.
Fixes #49249. See #42139, #37453.