'width' and 'height' HTML placeholder attributes in image dialog are not translated, yet should be lcfirst

While I realize our general standard is to "Capitalize" all strings in the interface, in this case I actually think it would add too much visual emphasis. Not sure. Tagging for the UX team to weigh in on.

"Needs usability review" would be the right tag I guess.

Issue summary updated to refer to the issue where this was introduced: #1879120: Use Drupal-specific image and link plugins — use core dialogs rather than CKEditor dialogs, containing alterable Drupal forms.

#1932652: Add image uploading to WYSIWYGs through editor.module landed, now let's get this fixed.

I have no particular preference, how do we lead in general with placeholder text capitalization?

#5 can you rephrase that? I don't understand the usage of the verb "lead" in that phrase.

As in, what do we commonly use with or without capital?

The only core uses of #placeholder at present are:

core/modules/block/lib/Drupal/block/BlockListController.php:      '#placeholder' => t('Filter by block name'),
core/modules/editor/editor.admin.inc:    '#placeholder' => 'width',
core/modules/editor/editor.admin.inc:    '#placeholder' => 'height',
core/modules/editor/lib/Drupal/editor/Form/EditorImageDialog.php:      '#placeholder' => 'width',
core/modules/editor/lib/Drupal/editor/Form/EditorImageDialog.php:      '#placeholder' => 'height',
core/modules/simpletest/lib/Drupal/simpletest/Form/SimpletestTestForm.php:      '#placeholder' => $this->t('Enter test name…'),
core/modules/system/lib/Drupal/system/Form/ModulesListForm.php:      '#placeholder' => $this->t('Enter module name'),
core/modules/system/tests/modules/form_test/form_test.module:      '#placeholder' => 'placeholder-text',

So we do not have any existing one-word examples other than this and a test module.

I agree with webchick that lowercase seems more aesthetically pleasing and less visually prominent here.

Ok, then I think we have a decision, no uppercase.

Patch attached, should the field suffix 'pixels' also get translated?

As well should there be another issue to add it to these, based on #8:

core/modules/editor/editor.admin.inc:    '#placeholder' => 'width',
core/modules/editor/editor.admin.inc:    '#placeholder' => 'height',

#11: translate-placeholder-2067119-11.patch queued for re-testing.

Yes, the "pixels" suffix should also be translated.

Thanks so much pameeela for pushing this forward!

(Queued for retesting because it was probably a random failure.)

Let's translate editor.admin.inc at the same time, as the code is almost identical. This patch adds that and also translates the 'pixels' suffix.

+    '#placeholder' => t('height'),
+    '#field_suffix' => t('pixels'),

Should it be $this->t() or does t() suffice? May not make a difference but thought I'd check.

If no difference then this is good for RTBC.

editor.admin.inc is a procedural file and $this is not available, so we use plain old t() there.

Yay, thanks :)

#16: 2067119-translate-image-placeholders-16.patch queued for re-testing.

Committed and pushed to 8.x. Thanks!