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 #1116 (closed enhancement: wontfix)

Opened 11 years ago

Last modified 8 years ago

[PATCH] for cleaner ListForm and TableForm markup

Reported by: nerkles Owned by: anonymous
Priority: normal Milestone: 1.x
Component: TG Widgets Version:
Severity: normal Keywords: widgets forms
Cc:

Description

This is to allow some more semantic and simpler uses of CSS for styling forms.

Ex:

.required label { font-weight: bold; }
.for_checkbox label { font-style: italic; }
.required input { background: #ffc; }

etc...

It also removes the class="fieldlabel" from labels because I just couldn't think of a case where that would be necessary.

Attachments

css.patch Download (1.6 KB) - added by nerkles 11 years ago.

Change History

Changed 11 years ago by nerkles

comment:1 Changed 11 years ago by jorge.vargas

please make a post in turbogears-trunk this is "an api change" and should be discussed before getting into 1.0

comment:2 Changed 11 years ago by godoy

With regards to fieldlabel applicability, now that we can select stuff on JavaScript? by applying some CSS rules this might become interesting. I don't have any use of that -- that I remember of -- as well, but I don't see any harm on it being there.

I liked moving the "requiredness" to the row as well. Maybe we could adopt that for other cases (what would make it much simpler to hide and show parts of the form conditionally to some previous selection made).

I also think that this should go on both 1.0 and the trunk if it gets in.

comment:3 Changed 11 years ago by alberto

  • Milestone changed from 1.0 to 1.1

comment:4 Changed 11 years ago by john.m.camara

  • Summary changed from patch for cleaner ListForm and TableForm markup to [PATCH] for cleaner ListForm and TableForm markup

modify summary to include [PATCH]

comment:5 Changed 11 years ago by alberto

  • Milestone changed from 1.1 to __unclassified__

Batch moved into unclassified from 1.1 to properly track progress on the later

comment:6 Changed 9 years ago by jorge.vargas

  • Milestone changed from __unclassified__ to 1.x

comment:7 Changed 8 years ago by Chris Arndt

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

Cruft ticket and I don't really see the use in this. Widget orms are perfectly stylable with CSS as is and you can alwasys overwrite the form template if needed. Closing ticket as wontfix.

Note: See TracTickets for help on using tickets.