Summary: | can select multiple layouts to edit | ||
---|---|---|---|
Product: | Koha | Reporter: | Nicole C. Engard <nengard> |
Component: | Label/patron card printing | Assignee: | Chris Nighswonger <cnighswonger> |
Status: | In Discussion --- | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | P5 - low | CC: | gmcharlt, kyle.m.hall, veron, wizzyrea |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
URL: | cgi-bin/koha/labels/label-manage.pl?label_element=layout | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Bug Depends on: | |||
Bug Blocks: | 7386 | ||
Attachments: |
Bug 7032 - can select multiple layouts to edit
Mockup of an alternate layout |
Description
Nicole C. Engard
2011-10-14 18:02:46 UTC
Created attachment 8776 [details] [review] Bug 7032 - can select multiple layouts to edit Switch from checkboxes to radio buttons. (In reply to comment #1) > Created attachment 8776 [details] [review] > Bug 7032 - can select multiple layouts to edit > > Switch from checkboxes to radio buttons. Personally I'd like to see these stay check boxes for two reasons: 1. This allows for multiple template deletes which was the original intent though it dropped through the cracks and did not get implemented. 2. Visual coordination and consistency with the rest of the label/patron creation interface. It should be just a matter of adjusting the JS trapping as the multiple delete code is already in the back-end IIRC (its been a while since I looked at it). (In reply to comment #2) > > It should be just a matter of adjusting the JS trapping as the multiple > delete code is already in the back-end IIRC (its been a while since I looked > at it). Actually it will also require a few moments in labels/label-manage.pl to mod the delete option to take multiple params and loop through them, deleting each one. http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=labels/label-manage.pl;h=c70ea6cfd3e5a68aec9f24ad2e49df539af0832e;hb=HEAD#l78 I would agree, but since this system does not support editing multiple templates at once, that would leave this issue unresolved. The best resolution would be to use "Edit" links for each row and leave the check-boxes for deleting only, but this is made practically impossible be the hideous way this system builds the tables. It would need a huge amount of rewriting to do so. (In reply to comment #4) > I would agree, but since this system does not support editing multiple > templates at once, that would leave this issue unresolved. The best > resolution would be to use "Edit" links for each row and leave the > check-boxes for deleting only, but this is made practically impossible be > the hideous way this system builds the tables. It would need a huge amount > of rewriting to do so. I must agree with your analysis of how the tables are built. It was quite a chore getting it written that poorly, I assure you. ;-) At any rate, my objection to the proposed fix stands for the reasons stated. I'll defer to QA and the RM, however, as I don't have time to fix it myself. FWIW, there are several libraries who use this feature and none has complained of this "problem." Created attachment 8788 [details] Mockup of an alternate layout > Personally I'd like to see these stay check boxes for two reasons: > > 1. This allows for multiple template deletes which was the original intent > though it dropped through the cracks and did not get implemented. This interface came up during our discussions at Hackfest of inconsistencies in Koha's interfaces. The most common pattern is for tables like this to have edit and/or delete links in each row for single-item operations. The column of checkboxes would then be used for bulk-delete operations. I've attached a mockup of what I'm thinking of. I think it is worthwhile to preserve the option of multiple deletions, but we shouldn't let that stop us from dealing with the ambiguity of having it appear as if one can select multiple lines to edit. (In reply to comment #6) > Created attachment 8788 [details] > Mockup of an alternate layout > > > Personally I'd like to see these stay check boxes for two reasons: > > > > 1. This allows for multiple template deletes which was the original intent > > though it dropped through the cracks and did not get implemented. > > This interface came up during our discussions at Hackfest of inconsistencies > in Koha's interfaces. The most common pattern is for tables like this to > have edit and/or delete links in each row for single-item operations. The > column of checkboxes would then be used for bulk-delete operations. I've > attached a mockup of what I'm thinking of. I think it is worthwhile to > preserve the option of multiple deletions, but we shouldn't let that stop us > from dealing with the ambiguity of having it appear as if one can select > multiple lines to edit. That looks nice. I'm not sure how hard it would be to implement, though. What Kyle says about the table-building code is very true... its ugly. I approve of Owen's suggestion - I'd like to see them consistent with other tables. |