Page MenuHomePhabricator

CodexHTMLForm: Codexify buttons at the bottom of the form
Closed, ResolvedPublic2 Estimated Story Points

Description

The buttons at the bottom of the form are generated by HTMLForm::getButtons(). We need to override this method to produce Codex buttons instead of plain buttons.

When the $wgUseCodexSpecialBlock = true; configuration is set to true in LocalSettings.php, the Codex version of the form (CodexHTMLForm) is produced.

Acceptance Criteria

  • Stop hard-coding button flags in HTMLButtonField T359018
  • Create a function getButtons() in CodexHTMLForm that produces Codex buttons at the bottom of the form.
    • As discussed, we will not include a Codex reset button because the reset button will be removed from HTMLForm.

Follow-up tasks

  • Remove the reset button from HTMLForm since it's unused. T361032
  • Refactor CodexHTMLForm::getButtons() to use a static method (similar to HTMLTextField).

Event Timeline

Catrope renamed this task from CodexHTMLForm: Codexify buttons at the bottom of the form in to CodexHTMLForm: Codexify buttons at the bottom of the form.Mar 4 2024, 6:59 PM
CCiufo-WMF removed a project: Epic.
lwatson set the point value for this task to 2.Mar 18 2024, 5:19 PM
lwatson changed the task status from Open to In Progress.Mar 20 2024, 9:32 PM
lwatson claimed this task.
lwatson changed the status of subtask T359018: HTMLButtonField: Don't hard-code button flags in Codex implementation from Open to In Progress.

Change #1013568 had a related patch set uploaded (by LWatson; author: LWatson):

[mediawiki/core@master] [WIP]CodexHTMLForm: Codexify buttons at the bottom of the form

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

Change #1013568 merged by jenkins-bot:

[mediawiki/core@master] CodexHTMLForm: Codexify buttons at the bottom of the form

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

Change #1016054 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/core@master] CodexHTMLForm: Make form submit button container a `div`

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

The container needs to be a block-level element, otherwise we can use the margins just for primping our hair. (A German idiom ;) )
That's also what it is in OOUIHTMLForm.

Change #1016054 merged by jenkins-bot:

[mediawiki/core@master] CodexHTMLForm: Make form submit button container a `div`

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

Change #1015615 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/core@master] htmlform: Introduce CodexHTMLForm stylesheet

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