#653: Create a tab widget
-------------------------+--------------------------------------------------
Reporter: kevin | Owner: anonymous
Type: enhancement | Status: new
Priority: normal | Milestone:
Component: Widgets | Version:
Severity: normal | Resolution:
Keywords: |
-------------------------+--------------------------------------------------
Comment (by alberto):
Nice widget! Fast, and with tests... :) Just a few comments:
* Shouldn't Tab be a container widget too? So you can nests tabs as the
example at barelyfitz shows...
* In Tab too, you can strip the whole __init__ method as all it does is
self.title = title, template_vars already handles that automatically (see
display() @ base.Widget ).
* Maybe MasterTab should be Tabber? (just to keep the original semantics)
* We've somehow informally decided that the only positional arguments
that widget's should receive is the "name" parameter as it's the most
frequently used one, ie: Form(fields=[TextField("name"),
TextField("age")]), maybe the "title" param at Tab should follow this
convention too.
Gonna give it a closer look when I get to work as I'm working on a gui now
and this looks very promising...
Thanks, Great work! :)
Alberto
--
Ticket URL: <http://trac.turbogears.org/turbogears/ticket/653>
TurboGears <http://www.turbogears.org/>
TurboGears front-to-back web development
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"TurboGears Tickets" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/turbogears-tickets
-~----------~----~----~----~------~----~------~--~---
