When editing, the text in the grammatical features field is now right-aligned instead of left-aligned.
e.g.
After clicking edit on https://www.wikidata.org/wiki/Lexeme:L123#F61:
After clicking into the field:
Nikki | |
Mar 26 2021, 9:33 AM |
F34396257: image.png | |
Apr 14 2021, 10:19 PM |
F34392059: image.png | |
Apr 14 2021, 4:06 PM |
F34191610: gf2.png | |
Mar 26 2021, 9:33 AM |
F34191608: gf1.png | |
Mar 26 2021, 9:33 AM |
When editing, the text in the grammatical features field is now right-aligned instead of left-aligned.
e.g.
After clicking edit on https://www.wikidata.org/wiki/Lexeme:L123#F61:
After clicking into the field:
Project | Branch | Lines +/- | Subject | |
---|---|---|---|---|
mediawiki/extensions/WikibaseLexeme | master | +2 -0 | Call the parent on onChangeTags as well |
Tentatively tagging OOUI, since I don’t think we’ve touched the relevant code in WikibaseLexeme recently.
The last OOUI update was in 1.36.0-wmf.35, and apparently included fixes for two TagMultiselectWidgets issues (T277177 and T230066), and Wikidata is currently on wmf.35 and still has the issue, so I think it’s likely that this is caused by some OOUI changes. (That’s not to say that I’m blaming OOUI – maybe we’re doing something in WikibaseLexeme that has always been incorrect but just happened to work out so far.)
Unclaiming this until I actually have my dev-environment in a state where I can work on it
To make it even more fun, in RTL languages it's left-aligned
Also, shouldn't the gloss editor have a bit of margin? Anyway...
This is very likely caused by changed in ooui (there are a lot of them touching position of input element recently in TagMultiselectWidget) (T276483: TagMultiselectWidget always puts input on separate line after line 1 for example)
The problem is that the input element is before the options while it should be the opposite:
I don't know what has caused it to behave like that (the easy ones look fine) yet but I'm investigating.
It's a bit hard to test it but reverting https://gerrit.wikimedia.org/r/c/oojs/ui/+/668488 would fix the issue. I'm sure this is an upstream problem
I can confirm that reverting to the commit right before https://gerrit.wikimedia.org/r/c/oojs/ui/+/668488 and using those build artifacts in mediawiki fixes the bug described in this issue.
This looks wrong:
/** * @protected */ onChangeTags: function () { this.updateInputPlaceHolder(); },
There is no call to the parent method.
Change 680306 had a related patch set uploaded (by Ladsgroup; author: Ladsgroup):
[mediawiki/extensions/WikibaseLexeme@master] Call the parent on onChangeTags as well
Change 680306 merged by jenkins-bot:
[mediawiki/extensions/WikibaseLexeme@master] Call the parent on onChangeTags as well