Change 349274 had a related patch set uploaded (by Jforrester):
[operations/mediawiki-config@master] Enable mobile non-JavaScript editing on all MobileFrontend wikis

Change 349114 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Reveal login/logout buttons when non-js editing is available

Isn't editpage-copywarn a MediaWiki message provided by the wiki?

Change 349114 had a related patch set uploaded (by Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Reveal login/logout buttons when non-js editing is available

Two nitpicks for the future:

Change 347127 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Improve layout of fallback editor

Looks good; merged your patches (except for the hygiene one which has a blip).

Change 347126 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Lead section edit icon should be visible when no-js editing is enabled

Can anyone in your team provide code review here? My team have not been involved in this work and under the reading banner I'm going to struggle to get code review support :)

We may want to consider hiding the minor edit and watch this page boxes. With OOjs UI the buttons get pushed below the fold and I don't think they are vital to the editing experience. I would hazard a guess that a small % of our editors actually use those boxes on desktop and in the current mobile editor we don't surface them.

So I've made some changes above to compact the information into the space available. Logged in users will see the buttons at the bottom of the page. Anonymous users however will need to scroll past the warning box which I assume is the intention :)

Change 347127 had a related patch set uploaded (by Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Improve layout of fallback editor

Change 347126 had a related patch set uploaded (by Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Lead section edit icon should be visible when no-js editing is enabled

In T125174#3155691, @Jdlrobson wrote:

That edit icon is easily revealed. In the existing view, we have nothing to link to hence why it's there. It can be displayed as part of wrapping up the work here.

That edit icon is easily revealed. In the existing view, we have nothing to link to hence why it's there. It can be displayed as part of wrapping up the work here.
After T111088 is applied it seems the main issues relate to use of space - seems easy enough to recover the space between heading and text area, shrink license size (I don't see why this needs legal sign off as they already signed off on the license text size in the editor overlay). Also I wonder if the heading "Editing Foo (section)" really need to be heading font-size. This doesn't seem like much work, if we wanted to spend a day focusing on this I think we could resolve these problems promptly.

In T125174#3120112, @Jdlrobson wrote:

The save buttons are not in view on open, but they're also never going to be as soon as the keyboard opens, so I'm not too worried.

Please do worry!! :) We've found buttons hidden below the fold to be less discoverable. We hit this issue with Special:MobileOptions - saves there halved when we accidentally pushed it below the fold. Most mobile interfaces I see surface the save buttons at the top of the screen on mobile and I'd suggest we'd be wise to do that - imo that's the only blocker here and there's plenty of space under the header to move them to be consistent with other products/our own!

Checking in, the current state of reality if you enable non-JS editing (simulated iPhone 5, which is already generous for low-end):

Remember this is not just non-JS editing. Slow connections may hit this page accidentally so I suggest that we give them a good experience too.

Checking in, the current state of reality if you enable non-JS editing (simulated iPhone 5, which is already generous for low-end):

Thanks to all of you for your help. :)

Change 335030 merged by Gedda:
Amend name for Goan Konkani

In T126148#3005647, @The_Discoverer wrote:

Does anyone have any information regarding this file?

Progress report
3 out of the 4 patches have been merged.
The only remaining patch is https://gerrit.wikimedia.org/r/335030
Does anyone have any information regarding this file?