Bug 14667

Summary: Label creator UI/UX revamp
Product: Koha Reporter: Liz Rea <liz>
Component: Label/patron card printingAssignee: Liz Rea <wizzyrea>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: cnighswonger, gwilliams, jonathan.druart, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on:    
Bug Blocks: 14718, 15072    
Attachments: Bug 14667 - UI/UX improvements for the Label creator
[Signed off] Bug 14667 - UI/UX improvements for the Label creator
Bug 14667 - UI/UX improvements for the Label creator
Bug 14667: Make some strings translatable
Bug 14667: (QA followup) make the UI show 'Label creator' consistently
Bug 14667: (QA followup) make the UI show 'Label creator' consistently
Bug 14667: (QA followup) remove useless labels-menu.inc file

Description Liz Rea 2015-08-10 09:30:18 UTC
The label creator is overdue for some UI/UX attention. This bug will keep track of the proposed changes.
Comment 1 Liz Rea 2015-08-10 10:24:30 UTC Comment hidden (obsolete)
Comment 2 Chris Nighswonger 2015-08-12 19:04:44 UTC
Nice improvements, Liz. I'd like to see these applied to the patron card tool in conjunction with this to keep the interfaces similar in look and feel.
Comment 3 Chris Nighswonger 2015-08-12 19:32:13 UTC Comment hidden (obsolete)
Comment 4 Liz Rea 2015-08-12 22:35:16 UTC
Hi Chris,

That is a good point, I'll put it on my list of pet projects, because yeah, that part needs updating too. There are lots of places that need love. :)

I wonder, could you help out with one thing though - the table headers are untranslatable at the moment, if you have some time could you have a look at that, or give me some hints as to how that could be done in a translatable way?

Liz
Comment 5 HB-NEKLS 2015-08-12 22:45:25 UTC
I'll be taking a look at this -- because this area definitely needed some love. And +1 on overhauling the patron card creator, including taking on bug 12804, please! 

Looking forward to looking at it, Liz.
Comment 6 Chris Nighswonger 2015-08-13 13:11:43 UTC
(In reply to Liz Rea from comment #4)
> Hi Chris,
> 
> That is a good point, I'll put it on my list of pet projects, because yeah,
> that part needs updating too. There are lots of places that need love. :)

And don't forget manual re-writes to match the new interface... ;-)

> 
> I wonder, could you help out with one thing though - the table headers are
> untranslatable at the moment, if you have some time could you have a look at
> that, or give me some hints as to how that could be done in a translatable
> way?

I implemented DT in both of these sections before (IIRC) it was implemented in Koha generally. I'm not sure what mechanism is currently used, but probably the code would need to be refactored to match what's presently being done other places. I'm pretty much unfamiliar with how that is, so I'll not be much help there.
Comment 7 Jonathan Druart 2015-08-18 16:41:38 UTC
Created attachment 41625 [details] [review]
Bug 14667 - UI/UX improvements for the Label creator

Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 8 Jonathan Druart 2015-08-18 16:41:44 UTC
Created attachment 41626 [details] [review]
Bug 14667: Make some strings translatable

This patch 1/ Fixes a regression introduced by the previous patch: the
"Are you sure to delete..." strings was translatable and 2/ Make the
table headers translatable.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 9 Jonathan Druart 2015-08-18 16:43:23 UTC
Liz, don't you think the help on the label home page was helpful ("Welcome to Koha's label creator module")?
Comment 10 Liz Rea 2015-08-18 21:12:16 UTC
I didn't - the reason I didn't is because it is not like any other page. That help should be in the upper right hand help menu, in my opinion.

Also thank you very much for helping out with the translation stuff. :)

Cheers,
Liz
Comment 11 Chris Nighswonger 2015-08-18 22:12:43 UTC
On the other hand, I thought it was cool and that every tool should have a solid intro page which gives a brief overview of its purpose and usage. But that's just my opinion. And pretty biased at that since I created it. ;-)
Comment 12 Liz Rea 2015-08-18 22:28:37 UTC
:) I can totally understand where you're coming from there.

I have observed, when watching librarians learn this tool, that they just don't read it, and the wall of text distracts them from what they are supposed to do. Possibly though, that's because I emphasize the help in the upper right hand corner so they get it drilled into them to look there for help.

Certainly the decision to remove it isn't scientific by any stretch, just based on my observations of people using the tool. :)
Comment 13 Tomás Cohen Arazi (tcohen) 2015-10-06 14:39:45 UTC Comment hidden (obsolete)
Comment 14 Tomás Cohen Arazi (tcohen) 2015-10-06 15:55:51 UTC
Created attachment 43176 [details] [review]
Bug 14667: (QA followup) make the UI show 'Label creator' consistently

The tool is documented and accessed (via Tools >) as 'Label creator'. But the
pages titles say 'Labels' and the breadcumbs 'Labels home'. It should be called
'Label creator' for consistency. This patch changes the tt files so they are
consistent.

It also makes the title show the same page name as the breadcumbs.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 15 Tomás Cohen Arazi (tcohen) 2015-10-06 15:56:02 UTC
Created attachment 43177 [details] [review]
Bug 14667: (QA followup) remove useless labels-menu.inc file

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 16 Tomás Cohen Arazi (tcohen) 2015-10-06 16:20:46 UTC
Patches pushed to master.

Good job Liz!
Comment 17 Liz Rea 2015-10-06 20:21:39 UTC
Oh awesome,thanks for the followups!