Add a base class (or mixin) for working with forms
rsyring opened this issue · comments
Randy Syring commented
Presumably, they created that as a mixin instead of a base class for the flexibility. We might want to provide both, with the base class using the mixin. If we do that, maybe we should do the same for the GridBase?
Randy Syring commented
@guruofgentoo you up for changing the FormMixin
a bit with some of the changes I made here: https://github.com/level12/brecham-marshal/blob/master/marsh/libs/views.py#L37
- A
form_create()
makes it easier to override just that functionality, which I found helpful in Marshal. - rename the "on" methods so they start with "form_" so that all form mixin functionality that isn't get/post have the same prefix.
- use "page_title" vs "title" to be more specific?
Matt Lewellyn commented
Yep, I'll get that done here shortly.