Code

Opened 2 years ago

Closed 2 years ago

#17152 closed Bug (fixed)

WizardView template documentation incorrect

Reported by: Bradley Ayers <bradley.ayers@…> Owned by: nobody
Component: contrib.formtools Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

https://docs.djangoproject.com/en/dev/ref/contrib/formtools/form-wizard/#creating-templates-for-the-forms has sample HTML showing how a wizard could be rendered.

However the this code is missing type="submit" on the buttons.

<button name="wizard_prev_step" value="{{ wizard.steps.first }}">{% trans "first step" %}</button>
<button name="wizard_prev_step" value="{{ wizard.steps.prev }}">{% trans "prev step" %}</button>

Attachments (1)

17152.diff (2.8 KB) - added by Bradley Ayers <bradley.ayers@…> 2 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 2 years ago by julien

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Ready for checkin

comment:3 Changed 2 years ago by julien

  • Triage Stage changed from Ready for checkin to Accepted

Back to 'Accepted' as it needs more than just a trivial doc patch.

comment:4 Changed 2 years ago by Bradley Ayers <bradley.ayers@…>

I think this is also a bad example on how to "go back". If you have a form that has a file field, and you choose a file to upload, and then click "back", it's going to submit the form, meaning your file will be uploaded. Although there's no other way to do it using WizardView, I think this limitation should be made more obvious (and urge developers to use NamedUrlWizardView instead which allows you to just use a hyperlink to the previous form.)

Changed 2 years ago by Bradley Ayers <bradley.ayers@…>

comment:5 Changed 2 years ago by Bradley Ayers <bradley.ayers@…>

  • Has patch set

comment:6 Changed 2 years ago by Bradley Ayers <bradley.ayers@…>

I've incorporated #17153 into this patch.

comment:7 Changed 2 years ago by julien

  • Triage Stage changed from Accepted to Ready for checkin

comment:8 Changed 2 years ago by julien

  • Resolution set to fixed
  • Status changed from new to closed

In [17072]:

Fixed #17152 -- Fixed a few anomalies in the contrib.formtools documentation and templates. Thanks, Bradley Ayers.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.