Page MenuHomePhabricator

TemplateWizard: Only fallback to focus on 'add all' button if it actually exists
Closed, ResolvedPublic0.5 Estimated Story Points


(Followup to the bug found in

We have a focus method by which we try to set the focus initially, and later when fields are added/removed based on either the first available field or the 'add/remove all' button.

The problem, is that the add/remove button can be missing if the template has no fields or if all fields are mandatory.

In that case, we have nothing to focus on, and the focus method should bail.

To test this:

  1. Try to load an empty template into TemplateWizard

Example TemplateData:

	"params": {},
	"description": "An empty template without parameters"

Expected: No crash :)

Related Objects

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 458606 had a related patch set uploaded (by Mooeypoo; owner: Mooeypoo):
[mediawiki/extensions/TemplateWizard@master] Only focus on 'add/remove all' if it actually exists

Mooeypoo set the point value for this task to 0.5.
Mooeypoo moved this task from Ready to Needs Review/Feedback on the Community-Tech-Sprint board.

Change 458606 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Only focus on 'add/remove all' if it actually exists

Niharika moved this task from QA to Q2 2018-19 on the Community-Tech-Sprint board.