[wp-trac] [WordPress Trac] #42574: MediaElement upgrade causing JS errors when certain languages are in use e.g de_DE-formal

[WordPress Trac]

#42574: MediaElement upgrade causing JS errors when certain languages are in use
e.g de_DE-formal
-------------------------------------+-------------------------------------
 Reporter:  erich_k4wp               |       Owner:
     Type:  defect (bug)             |      Status:  reopened
 Priority:  high                     |   Milestone:  4.9.1
Component:  Media                    |     Version:  4.9
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch 2nd-opinion    |     Focuses:  javascript,
  needs-testing                      |  administration
-------------------------------------+-------------------------------------

Comment (by ocean90):

 Replying to [comment:45 SergeyBiryukov]:

 > [attachment:42574.3.diff] seems a bit too technical for translators.
 Without looking deep into MediaElement.js code, how are they supposed to
 know which language tags are supported and which are not? Unlike in
 [36802], there's no documentation link here.

 By writing our own documentation or submitting a PR which adds the missing
 documentation.


 [attachment:42574.5.diff] is basically using the fallback from
 [attachment:42574.3.diff] so that should be fine too. We could also drop
 the `strtolower()` call since the first part is always lowercased.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/42574#comment:48>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform