Bug 7032 - can select multiple layouts to edit
Summary: can select multiple layouts to edit
Status: In Discussion
Alias: None
Product: Koha
Classification: Unclassified
Component: Label/patron card printing (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Chris Nighswonger
QA Contact: Bugs List
URL: cgi-bin/koha/labels/label-manage.pl?l...
Keywords:
Depends on:
Blocks: 7386
  Show dependency treegraph
 
Reported: 2011-10-14 18:02 UTC by Nicole C. Engard
Modified: 2016-10-03 15:38 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 7032 - can select multiple layouts to edit (1.34 KB, patch)
2012-04-02 15:02 UTC, Kyle M Hall
Details | Diff | Splinter Review
Mockup of an alternate layout (16.55 KB, image/png)
2012-04-02 19:39 UTC, Owen Leonard
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nicole C. Engard 2011-10-14 18:02:46 UTC
I've noticed that I am allowed to check off more than one layout in the list of layouts.  It's not a big deal, since the system tells me to choose only one layout to edit, but it seems like those check boxes should be exclusive (I can't come up with the right term for it) -- so if I check one box and then check another box, the initial check should go away when I check box #2 or they should be radio buttons instead.
Comment 1 Kyle M Hall 2012-04-02 15:02:08 UTC
Created attachment 8776 [details] [review]
Bug 7032 - can select multiple layouts to edit

Switch from checkboxes to radio buttons.
Comment 2 Chris Nighswonger 2012-04-02 16:52:34 UTC
(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).
Comment 3 Chris Nighswonger 2012-04-02 16:57:51 UTC
(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
Comment 4 Kyle M Hall 2012-04-02 19:23:37 UTC
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.
Comment 5 Chris Nighswonger 2012-04-02 19:34:00 UTC
(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."
Comment 6 Owen Leonard 2012-04-02 19:39:09 UTC
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.
Comment 7 Chris Nighswonger 2012-04-02 19:46:25 UTC
(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.
Comment 8 Liz Rea 2012-04-10 15:12:19 UTC
I approve of Owen's suggestion - I'd like to see them consistent with other tables.