added the ability to flag a select as multiple-select #92

Merged
merged 2 commits into from May 2, 2012

Conversation

Projects
None yet
2 participants
Member

sontek commented May 2, 2012

No description provided.

sontek referenced this pull request May 2, 2012

Closed

Added the multiple attribute #67

Member

sontek commented May 2, 2012

deformdemo test for this is here: Pylons/deformdemo#7

@kiorky kiorky commented on an outdated diff May 2, 2012

deform/templates/select.pt
<select name="${field.name}"
id="${field.oid}"
tal:attributes="size field.widget.size"
- tal:attributes="class field.widget.css_class">
+ tal:attributes="class field.widget.css_class"
+ tal:attributes="multiple field.widget.multiple">
@kiorky

kiorky May 2, 2012

Member

Please do not double tal:attributes (use tal:a="id val;id val")

Member

kiorky commented May 2, 2012

Please add a test for multiple selects, i have it on a personal project and at first i broke the widget.

Member

kiorky commented May 2, 2012

Ho i didnt read all ! lookin the test

Member

kiorky commented May 2, 2012

Ok, fix double tal:a, & i merge !

Member

sontek commented May 2, 2012

I'm not a chameleon guy, have never used it before and don't know what you want, sorry!

Member

kiorky commented May 2, 2012

A big Thanks!

kiorky merged commit df64b3f into Pylons:master May 2, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment