It would be nice to understand the strategic direction with this featureegiven the many open bugs ^

So as this ticket is now open for one year without any activity: Would it be possible at least to fix the bug with the submit button leaving the view port? Or should I open a new ticket for that? I don't really like that I have to switch to desktop mode every time I want to edit image categories on mobile.

Possible solution (quick fix, not perfect):

.overlay-header {
  position: relative;
  max-width:100vw
}
.header-action {
  position: absolute !important;
  right: 0;
}
.overlay-title {
  max-width: 50vw;
}
.oo-ui-buttonElement-button {
  max-width: 90vw;
}

Result:

@Discostu the problem here is the code is not a high priority.
I'd happily review any fixes for the category feature if they were to be uploaded, but this bug is unlikely to get focus without a volunteer patch. I think changing the label from "Add to category" to "Add" would be the easiest solution here.

Change 683441 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Replace categories button with HTML rendering

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

Test wiki created on Patch Demo by Jdlrobson using patch(es) linked to this task:

https://patchdemo.wmflabs.org/wikis/bc383298ea/w/

@ovasileva: The description of this task and @Jdlrobson's (demo?) patch seem very-nearly-almost complete. Should this be moved across the board?

In T246049#7056628, @phuedx wrote:

@ovasileva: The description of this task and @Jdlrobson's (demo?) patch seem very-nearly-almost complete. Should this be moved across the board?

We're still discussing what the alternative design should be. There's currently some concerns from myself and @alexhollender on displaying the entire list if we were to do T152199: Display categories for logged-in users on mobile web stable as well. We could potentially:

• remove the feature from AMC and show the list for the time being
• wait on continuing with T152199: Display categories for logged-in users on mobile web stable until we have a final design (that we'd implement for AMC and all logged-in).

@alexhollender - would you be okay with showing the full list for AMC only just for the time being? If so, I think we can go ahead with this one.

Following Olga's recommendation in T152199 this is ready for dev.

Change 683441 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Replace categories button with HTML rendering

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

Change 699238 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/extensions/MobileFrontend@master] Remove categories feature

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

@Edtadros I forgot I have some follow up work here :)

Change 699238 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@master] Remove categories feature

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

@Jdlrobson I don't like so much the deletion of the Mobile Frontend Categories Feature, but i like so much.

Change 700700 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Category feature not ready for logged in users

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

Change 700700 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Category feature not ready for logged in users

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

Test Result - Beta

Status: ✅ Pass
Environment: beta
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA

Test Artifact(s):

QA Steps

Visit https://en.m.wikipedia.beta.wmflabs.org/wiki/Spain
✅ AC1: In logged in / AMC: There is a category box at the bottom of the page

✅ AC2: In logged in with AMC disabled: There is no category box at the bottom of the page.

✅ AC3: When logged out: There is no category box at the bottom of the page.

In T246049#7168128, @Jdlrobson wrote:

We talked about this in standup and agreed to go ahead with reverting the change. This will go out with the train but I can backport earlier if there's urgency here (Olga let me know!). If anyone complains we will prompt them to opt into the advanced mobile mode and direct them to T152199.

Thanks @Edtadros for catching in QA! This should pass now.

Train is okay

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

https://patchdemo.wmflabs.org/wikis/bc383298ea/w/