I am opening this bug to address so HTML markup errors on orderreceive.tt. -One of the <th> is closed with a </td> -The #jobpanel element div is not properly closed -Style tags are in the HTML body and they should be in the HTML head -The following is invalid - <div id="modal-order-main row"> It should be: + <div id="modal-order-main" class="row">
Created attachment 151429 [details] [review] Bug 33771: Fix markup errors in orderreceive.tt This patch fixed the following on orderreceive.tt: -One of the <th> is closed with a </td> -The #jobpanel element div is not properly closed -Style tags are in the HTML body and they should be in the HTML head -The #modal-order-main needs a class of 'row', this has been added. Apply the pacth and follow the steps in 8179 until you get to the orderreceive.tt. There should be no visual changes but the markup has been corrected.
Created attachment 151430 [details] [review] Bug 33771: Fix markup errors in orderreceive.tt This patch fixed the following on orderreceive.tt: -One of the <th> is closed with a </td> -The #jobpanel element div is not properly closed -Style tags are in the HTML body and they should be in the HTML head -The #modal-order-main needs a class of 'row', this has been added. Apply the pacth and follow the steps in 8179 until you get to the orderreceive.tt. There should be no visual changes but the markup has been corrected. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Pushed to master for 23.05. Nice work everyone, thanks!
Dependency missing - not backporting to 22.11.x