To fix a FIXME in the template: Add a "clearfix" class in the form markup so that the standard '<fieldset class="rows">' can be used. Also: Lengthen the title field and add a maxlength attribute to match the table column.
Created attachment 124939 [details] [review] Bug 29042: Improve formatting of entry form in Additional Contents This patch makes some corrections to the Additional Contents template so that the content entry form can be styled like other similar forms in the staff interface. The patch also updates the form's "title" field so that it is longer and has a maxlength attribute matching the size of the table column. To test, apply the patch and go to Tools -> News. - Create a new news item. - In the entry form, confirm that the "Title" and "Content" fields are styled consistently with other similar forms. - Confirm that the "title" field is longer.
Created attachment 125403 [details] [review] Bug 29042: Improve formatting of entry form in Additional Contents This patch makes some corrections to the Additional Contents template so that the content entry form can be styled like other similar forms in the staff interface. The patch also updates the form's "title" field so that it is longer and has a maxlength attribute matching the size of the table column. To test, apply the patch and go to Tools -> News. - Create a new news item. - In the entry form, confirm that the "Title" and "Content" fields are styled consistently with other similar forms. - Confirm that the "title" field is longer. Signed-off-by: David Nind <david@davidnind.com>
Almost sad to see this line removed :) <fieldset>[%# FIXME We should remove list-style:none; and use class="rows" here but it does not work. Owen please help! %]>
Created attachment 125655 [details] [review] Bug 29042: Improve formatting of entry form in Additional Contents This patch makes some corrections to the Additional Contents template so that the content entry form can be styled like other similar forms in the staff interface. The patch also updates the form's "title" field so that it is longer and has a maxlength attribute matching the size of the table column. To test, apply the patch and go to Tools -> News. - Create a new news item. - In the entry form, confirm that the "Title" and "Content" fields are styled consistently with other similar forms. - Confirm that the "title" field is longer. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 21.11, thanks to everybody involved!