Arachnid / bloggart

A blog application for App Engine
http://bloggart-demo.appspot.com/
287 stars 78 forks source link

Djangoforms should be removed #24

Closed sylvainvivien closed 15 years ago

sylvainvivien commented 15 years ago

I think Djangoforms should be removed for 2 reasons :)

1 - Personaly, (so not very important :p ), I think that djangoforms is overkill for only 3 text fields.

2 - Most important this time : Djangoforms goes against the second rule of bloggart "Good isolation of code and markup. Users shouldn't have to understand the innards of our blogging software if all they want to do is change how the blog looks or is laid out." and particulalry for 2 things :

So I think it could be a nice feature to remove Djangoforms and to use for example jquery to check the forms (mandatory fields) and add a nice alert box (jalert : http://abeautifulsite.net/notebook_files/87/demo/)... and maybe start a little Ajax (for the preview for example,...)

And example of jquery code to validate the form (it has to be checked server side too)

Githubissues.
  • Githubissues is a development platform for aggregating issues.