Page MenuHomePhabricator

Improve styling of the non-js editor in mobile so the form is usable
Closed, ResolvedPublic0 Estimated Story Points


Non-js editing was just enabled on Ptwiki ( but it looking terribly cluttered and not very usable.

Screen Shot 2017-07-05 at 11.13.36 AM.png (957×464 px, 100 KB)

Screen Shot 2017-07-05 at 11.13.41 AM.png (891×452 px, 160 KB)


  • Change the font size of the text at the top of the page (include it in the .action-submit .preview-limit-report-wrapper, .action-edit .preview-limit-report-wrapper, .action-submit .diff-otitle, .action-edit .diff-otitle, .action-submit .diff-ntitle, .action-edit .diff-ntitle, .action-submit #section_0, .action-edit #section_0, .action-submit #editpage-copywarn, .action-edit #editpage-copywarn, .action-submit #mw-anon-edit-warning, .action-edit #mw-anon-edit-warning css rule?)
  • Reconsider displaying .mw-editTools or at least make it less in-your-face.

Event Timeline

Including the other elements in the #mw-anon-edit-warning css rule and adding some specific styles to the top "warning" box results in:

Screen Shot 2017-07-05 at 11.26.38 AM.png (933×704 px, 121 KB)

but it still looks badly designed.
This looks terribly broken and unprofessional in current form.
We can do better @Niharika and @Jdforrester-WMF ...

I'm minded to decline these kind of changes in general. Ultimately if the community want this to look terrible, that's how it's going to appear. Trying to hack code around community-maintained interface content is just a race to the bottom; normally, we ask the community to reconsider their choices.

This is not community maintained content.

The edit notice is provided by core:

As are edit-tools

The mobile editor is not styling either correctly. I'm writing a patch...

Change 363643 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Minor modifications to core editing elements in non-js mode

Change 363643 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Minor modifications to core editing elements in non-js mode

So the remaining part seems to be a non-style-guide conforming warning message coming from the community. @Jdlrobson Resolve what we could resolve?

Jdlrobson claimed this task.

Yes I think the remaining work involves T166915 and T127405

I've had a go at communicating the change by the edit button in good faith:

I'm hoping that will be seen as an improvement. If not we'll at least know a little more and have the starting point of a conversation.

Jdforrester-WMF moved this task from To Triage to TR1: Releases on the VisualEditor board.
Jdforrester-WMF removed a project: Patch-For-Review.