type=search breaks internal linking spinner position

[nacin]

Some editor.css styling is applied to type=text, which breaks when we moved to type=search.

The spinner appears between the label and the search field there, making the search field jump right. 21351.patch moves the spinner on the left of the label (no jumping) and fixes the vertical alignment.

Don't see any differences in styling input[type="text"] and input[type="search"], all seem fixed.

In [21306]:

Fix internal linking search spinner position, fixes #21351

In [21307]:

Fix internal linking search spinner position by reverting the search field's type to "text", fixes #21351 for 3.4

Shouldn’t the spinner sit on the right side of the search field?

In the order of actual actions this would feel more … natural:

Search … for keywords … wait

results

That's where it was in 3.3.

All other similar places have the spinner to the left, but yes, it was to the right in the internal linking popup in 3.3. Will move it back.

In [21319]:

Put the spinner back to the right of the search field in internal linking, fixes #21351

In [21320]:

RTL css for the spinner in internal linking, see #21351