Bug 16048

Summary: Making notices actions buttons
Product: Koha Reporter: Aleisha Amohia <aleisha>
Component: NoticesAssignee: Aleisha Amohia <aleisha>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: brendan, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16036
Change sponsored?: Sponsored Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 16048: Making notices actions buttons
Bug 16048: Making notices actions buttons
[SIGNED-OFF] Bug 16048: Making notices actions buttons
Bug 16048 [Follow-up] Making notices actions buttons
Bug 16048: Making notices actions buttons
Bug 16048 [Follow-up] Making notices actions buttons

Description Aleisha Amohia 2016-03-10 22:46:04 UTC

    
Comment 1 Aleisha Amohia 2016-03-10 22:47:31 UTC Comment hidden (obsolete)
Comment 2 Aleisha Amohia 2016-03-10 23:46:52 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2016-03-11 07:42:49 UTC
Aleisha, it is better to define a class name for the td and set a style in the css file:

Something like:
<td class="actions">

.actions {
  white-space: nowrap;
}
Comment 4 Owen Leonard 2016-03-11 13:11:48 UTC Comment hidden (obsolete)
Comment 5 Owen Leonard 2016-03-11 13:11:52 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2016-03-11 13:17:48 UTC
Created attachment 49010 [details] [review]
Bug 16048: Making notices actions buttons

EDIT: Adding no wrap class to Actions column for when libraries with longer names mess up formatting

To test:
1) Go to Tools -> Notices and slips (tools/letter.pl)
2) Confirm that all actions (Copy, Edit and Delete) now show as font awesome buttons and work as expected

Sponsored-by: Catalyst IT
Signed-off-by: Owen Leonard <oleonard@myacpl.org>

I have a couple of corrections in a follow-up.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 7 Jonathan Druart 2016-03-11 13:17:54 UTC
Created attachment 49011 [details] [review]
Bug 16048 [Follow-up] Making notices actions buttons

This followup adds a "td.actions" style with "white-space:nowrap" and
replaces the inline styles. Also modified is the way non-sorting table
columns are defined. This corrects a problem with non-sortable columns
introducted by the previous patch's column changes.

To test, apply the patch and clear your browser cache if necessary.

- Go to Tools -> Notices and confirm that even at narrow browser widths
  the "Edit" and "Delete" buttons stay side by side.
- Confirm that all columns are sortable except "Copy notice" and
  "Actions."

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 8 Brendan Gallagher 2016-03-11 23:02:16 UTC
Pushed to Master - Should be in the May 2016 release.  Thanks