Created attachment 27675 [details] Invoices table In the table of invoices the last column is not wide enough for the links, that causes each line to be 3 lines.
It really looks ugly... My first idea was to put non breakting spaces around te slashes in lines 144 and 151 in invoices.tt, but this does not work out because of the formatting (indenting) with spaces. One would have to putt all of the if-else-logic on one line, and that is no good idea. Looking at the HTML code I see tat all column headers get a hardcoded style "width" with values between 19.2 and 126.2px - exept of the last one with the action links. But I do not see where this is done?
Created attachment 27762 [details] [review] Bug 12147 - Column width for action links on invoices too small To test: Got to Home > Acquisitions > Invoices and verify that the last column of invoices table displays properly and, in HTML code, has a property like "style="width: 134.2px;"
Created attachment 27949 [details] [review] Bug 12147 [Alternate] Column width for action links on invoices too small This alternative patch groups "action" links into a menu as was done with Bug 11763. This patch also converts the date column sorting configuration to use header class instead of an index. This patch also corrects a couple of HTML markup errors: An improperly-closed <input> and an improperly closed <option>. To test, view the invoices page and confirm that the "Actions" menu button works corectly. Test the functionality of each menu options: Details, Close, Reopen, and Delete. Confirm that sorting by billing date still works correctly.
Created attachment 27956 [details] [review] [Signed-off] Bug 12147 [Alternate] Column width for action links on invoices too small This alternative patch groups "action" links into a menu as was done with Bug 11763. This patch also converts the date column sorting configuration to use header class instead of an index. This patch also corrects a couple of HTML markup errors: An improperly-closed <input> and an improperly closed <option>. To test, view the invoices page and confirm that the "Actions" menu button works corectly. Test the functionality of each menu options: Details, Close, Reopen, and Delete. Confirm that sorting by billing date still works correctly. Much nicer user interface. Followed test plan. Works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
[% FOREACH budget IN budgets_loop %] - <option value="[% budget.budget_id %]"> - [% budget.budget_name %] + <option value="[% invoice.invoiceid %]">[% budget.budget_name %] Is this correct? invoice and budget..
Comment on attachment 27956 [details] [review] [Signed-off] Bug 12147 [Alternate] Column width for action links on invoices too small Review of attachment 27956 [details] [review]: ----------------------------------------------------------------- ::: koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt @@ +183,4 @@ > <select id="merge_shipment_budgetid" name="shipment_budget_id"> > <option value="">No fund</option> > [% FOREACH budget IN budgets_loop %] > + <option value="[% invoice.invoiceid %]">[% budget.budget_name %] Owen, could you confirm this change is intended?
Created attachment 28162 [details] [review] [Signed-off] Bug 12147 [Alternate] Column width for action links on invoices too small Bug 12147 [Alternate] Column width for action links on invoices too small This alternative patch groups "action" links into a menu as was done with Bug 11763. This patch also converts the date column sorting configuration to use header class instead of an index. This patch also corrects a couple of HTML markup errors: An improperly-closed <input> and an improperly closed <option>. To test, view the invoices page and confirm that the "Actions" menu button works corectly. Test the functionality of each menu options: Details, Close, Reopen, and Delete. Confirm that sorting by billing date still works correctly. Much nicer user interface. Followed test plan. Works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
(In reply to Jonathan Druart from comment #6) > Owen, could you confirm this change is intended? It was not. I have attached a corrected patch.
Created attachment 28167 [details] [review] Bug 12147: Column width for action links on invoices too small This alternative patch groups "action" links into a menu as was done with Bug 11763. This patch also converts the date column sorting configuration to use header class instead of an index. This patch also corrects a couple of HTML markup errors: An improperly-closed <input> and an improperly closed <option>. To test, view the invoices page and confirm that the "Actions" menu button works corectly. Test the functionality of each menu options: Details, Close, Reopen, and Delete. Confirm that sorting by billing date still works correctly. Much nicer user interface. Followed test plan. Works as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
The patch didn't applied, I got "fatal: cannot convert from UTF-8utf-8 to UTF-8". I applied it manually and recreated the path. Note there is no confirmation needed on deleting an invoice. Which is quite dangerous I think. This behavior existed before this patch. Marked as Passed QA.
Pushed to master and 3.16.x. Thanks, Owen!