Page MenuHomePhabricator

"There is no video associated with the current subtitle page" squeezed to one letter per line
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

What happens?:
The warning "There is no video associated with the current subtitle page" is horizontally squeezed to the point that each line has only one letter.

image.png (1×533 px, 44 KB)

What should have happened instead?:
The warning appears normally.

Other information:
The culprit appears to be:

.oo-ui-labelWidget {
	display: inline-block;
}

It's using a Codex box inside an OOUI element for some reason.

Event Timeline

likely caused by https://gerrit.wikimedia.org/r/1074283 (T360668), which removed non-codex output for the message box methods. One solution would be to switch from html::errorBox and html::warningBox to the OOUI messageWidget(Type: Warning) and MessageBox(Type: Error)

TheDJ added subscribers: Volker_E, TheDJ.

What the heck ? Interesting combo of options is in play here indeed.

The cdx-message-content has its width set to 0, with flex-basis: auto (which should thus default to 0) AND has break anywhere... Which makes it strive to be 0, which indeed requires applying as much line breaking as possible.
But if you remove the width:0 it works. If you remove the width AND set flex-basis: 0 it STILL works (which as far as I know, should be the exact same, but I guess not under some of the line breaking rules)

ping @Volker_E

CCiufo-WMF subscribed.

I don't understand why this message is inside of a label...it looks fine if you move the message outside of the label (even if it's still within the OOUI field):

image.png (352×1 px, 39 KB)

Maybe someone can fix the placement?

Change #1117979 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):

[mediawiki/extensions/TimedMediaHandler@master] TimedTextPage: Move error/warning box before the form

https://gerrit.wikimedia.org/r/1117979

Thanks for picking this up so instantly @matmarex!

Change #1117979 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] TimedTextPage: Move error/warning box before the form

https://gerrit.wikimedia.org/r/1117979

Test wiki on Patch demo by Matma Rex using patch(es) linked to this task was deleted:

http://patchdemo.wmcloud.org/wikis/a83096dc89/w/