https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... File MoinMoin/templates/blog/modify_entry_meta.html (right): https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... MoinMoin/templates/blog/modify_entry_meta.html:6: {# TODO: Make blog_entry nameless #} i'm not sure whether this should be moved https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... MoinMoin/templates/blog/modify_entry_meta.html:21: {% macro basic_meta_editor(form) %} they seem to be very similar, is it possible to reduce duplicates? https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... MoinMoin/templates/blog/modify_entry_meta.html:30: <div class="form-group"> how is forms.render() different from this div? https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... File MoinMoin/templates/blog/modify_main_meta.html (right): https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... MoinMoin/templates/blog/modify_main_meta.html:20: {% macro basic_meta_editor(form) %} same here.... the whole thing is duplicated (which is not related to this change) maybe it worth merging the code
https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... File MoinMoin/templates/blog/modify_entry_meta.html (right): https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... MoinMoin/templates/blog/modify_entry_meta.html:6: {# TODO: Make blog_entry nameless #} On 2014/07/16 12:52:51, dimazest wrote: > i'm not sure whether this should be moved I moved this as it was showing template syntax error
https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... File MoinMoin/templates/blog/modify_entry_meta.html (right): https://codereview.appspot.com/114850044/diff/1/MoinMoin/templates/blog/modif... MoinMoin/templates/blog/modify_entry_meta.html:30: <div class="form-group"> On 2014/07/16 12:52:52, dimazest wrote: > how is forms.render() different from this div? I tried using form.render it produces simple text areas without any bootstrap. By using the current method we can give a css class(bootstrap). Also I took reference from basic/templates/modify_meta.html and it was done the same way in that template