Page MenuHomePhabricator

Translation editor shows a gap between header and content
Closed, ResolvedPublic

Description

ContentTranslation editor has a header with options above the translation content, currently there seems to be an unnecessary gap that pushes content down and makes the tool area look disconnected:

Screen Shot 2017-08-25 at 11.04.33.png (736×1 px, 266 KB)

The cause seems to be a <fieldset> element:

Screen Shot 2017-08-25 at 11.04.28.png (537×1 px, 161 KB)

Either the empty fieldset should not have any height (display: none; for example) ot not be there at all when there is message.

Event Timeline

Nikerabbit subscribed.

This seems quite simple task which I can mentor. Either the empty fieldset should not have any height (display: none; for example) ot not be there at all when there is message.

divadsn subscribed.

I think it's better to remove the element completely in my opinion :)

Sorry, but sadly I can't reproduce the bug on the Wikipedia ContentTranslation page and my local vagrant had problems running the contenttranslation role.

Please check my screenshot below.

grafik.png (884×1 px, 254 KB)

If it comes to the gap between, the cause isn't the fieldset anymore, it's a wrong CSS rule here:

grafik.png (899×1 px, 275 KB)

Change 398691 had a related patch set uploaded (by Divadsn; owner: Divadsn):
[mediawiki/extensions/ContentTranslation@master] Fix gap between header and content

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

Change 398691 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] Fix gap between header and content

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

Thanks to @santhosh for his help in reviewing the task :)

I will need to check it in production to make sure that the additional messages can be displayed there.
wmf.12

Screen Shot 2017-12-18 at 4.09.17 PM.png (470×1 px, 169 KB)

Checked per my previous comment - in testwiki (wmf.15) I saw that the additional message does not quite fit anymore:

Screen Shot 2018-01-02 at 4.11.57 PM.png (551×1 px, 220 KB)

Screen Shot 2018-01-02 at 4.11.57 PM.png (551×1 px, 220 KB)

@divadsn - can your fix accommodate the case when there is some additional messages need to be displayed?