Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #2292 (closed defect: fixed)

Opened 8 years ago

Last modified 8 years ago

[Patch] Form widgets should have empty string for default action instead of None

Reported by: PeterRussell Owned by:
Priority: normal Milestone: 1.0.x bugfix
Component: TurboGears Version: 1.0 HEAD
Severity: normal Keywords: patch
Cc:

Description

Just a little HTML compliance niggle:

If you display a form without passing an action parameter, you get invalid HTML as Kid strips attributes which are None. What you probably want in these cases is an empty URI, to cause the UA to send the form to the URL that it came from.

Attached is a patch to do this.

Attachments

form.patch Download (480 bytes) - added by PeterRussell 8 years ago.

Change History

Changed 8 years ago by PeterRussell

comment:1 Changed 8 years ago by chrisz

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

Makes sense. Applied in r6629.

Note: See TracTickets for help on using tickets.