Bug 18042 - Update buttons in batch record modification and deletion
Summary: Update buttons in batch record modification and deletion
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Tools (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement with 10 votes (vote)
Assignee: Aleisha Amohia
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 17873
  Show dependency treegraph
 
Reported: 2017-02-02 23:43 UTC by Aleisha Amohia
Modified: 2023-09-10 12:02 UTC (History)
5 users (show)

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


Attachments
Bug 18042: Updating buttons in batch record modification and deletion (3.45 KB, patch)
2017-02-02 23:44 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 18042: Updating buttons in batch record modification and deletion (3.51 KB, patch)
2017-02-06 22:51 UTC, Aleisha Amohia
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Aleisha Amohia 2017-02-02 23:43:10 UTC
Buttons should use bootstrap 3
Comment 1 Aleisha Amohia 2017-02-02 23:44:43 UTC
Created attachment 59836 [details] [review]
Bug 18042: Updating buttons in batch record modification and deletion

To test:
Create a test record that you can edit and delete. Go through each
process of modifying the record, then deleting the record - once before
applying the patch, and after apply the patch. Notice the difference in
buttons and confirm they all still work as expected.

Sponsored-by: Catalyst IT
Comment 2 Josef Moravec 2017-02-03 09:31:55 UTC
I am for fa icon on Delete and Modify buttons,

for submit buttons we should probably start using btn-primary class, as Owen used in his restyling webinstaller work. 

ccing Owen to have his comment here, thanks ;)
Comment 3 Aleisha Amohia 2017-02-06 22:51:13 UTC
Created attachment 59959 [details] [review]
Bug 18042: Updating buttons in batch record modification and deletion

To test:
Create a test record that you can edit and delete. Go through each
process of modifying the record, then deleting the record - once before
applying the patch, and after apply the patch. Notice the difference in
buttons and confirm they all still work as expected.

Sponsored-by: Catalyst IT
Comment 4 J Schmidt 2017-02-09 16:41:44 UTC
Tested bug #18042 on 2/9/2017 using ByWater Sandbox. Steps taken:
1.) Koha Cataloging > New Record on Default Framework. 
2.) Using the simple editor, filled in these MARC fields: 000, 005, 006, 007, 008, 020, 100, 245, 300, 650 tied to one authority record, 942$c for BOOK.
3.) Saved the new record. Koha prompted me to fill in the required fields: 003 and 040. Then saved
4.) Koha prompts me to create a new item (Add item). Fields filled in include: 952$3, $8, $d, $p, $y, $t, $v. Add item was successful.
5.) Test record is now done. #439.
6.) Search to find two more bib numbers: #171, #272.
7.) Koha > Tools > Batch Record Modification. Received the message: " No MARC modification template is defined. You have to create at least one template for this tool. 
8.) Click Manage MARC Modification Templates. Message here is "There are no defined templates. Please create a template first.
9.) Tools>MARC Modification templates > New Template created. Add in New Action. I like the new buttons. 
10.) Under Add a new action, Add/Update 942$n  with value 1. Add action. 
11.) Add in a second action to new template. Add/update 650 x with value Massachusetts. Add Action. 
12.) Click on Batch Record Modification. Selected option to enter a list of three bib numbers: 439, 171, 272. Select modification template and Continue.
13.) Next screen gives me a new area where users can still DESELECT individual records which is nice.  There is a SHOW MARC button which is also nice. Show MARC does pop up dialog with MARC fields....
14.) I clicked Modify selected records. I receive a message that the three bibs were successfully modified. Prompted to do a new batch record modification..... 
15.) I click on record #171. I go to edit bib mode. I see 942$n has a 1. 650 has a $x for Massachusetts. 
16.) I  click on Batch record deletion under Tools.
17.) I enter in a list of three bib numbers: 439, 171, 272. I click Continue. Buttons look clean.
18.) Next screen for batch record deletion gives me the option to deselect individual records; select all, clear all, select without holds, select without items.  I clicked Delete selected records. Success. Prompted to do another batch record deletion. 
19.) I search for deleted bibs but they are removed.  

Functionality and buttons work well. jls. 
19.
Comment 5 Owen Leonard 2017-02-14 13:38:42 UTC
(In reply to Josef Moravec from comment #2)
>
> for submit buttons we should probably start using btn-primary class, as Owen
> used in his restyling webinstaller work. 

Thanks for asking Josef. I usually don't catch Bugzilla CC's so if you think of it @later me on IRC with questions.

I really think we need to come up with a comprehensive plan for updating buttons. As I commented on Bug 17873, I think we need to get proposals for how different kinds of buttons are going to be updated:

https://wiki.koha-community.org/wiki/Interface_patterns#Buttons

My initial thoughts about this process:

- I think we should come up with updates to the default Bootstrap button styles if we are going to use more than the btn-default class. Bootstrap colors aren't designed to work with Koha's, and we should avoid looking like out-of-the-box Bootstrap.

- I don't think we should add icons to all buttons. I think the icons are useful in places where there are multiple buttons which need to be distinguished from each other. I think submit buttons all serve a single basic purpose which doesn't require an added icon.

Those are just my opinions of course. I think we should work together to come up with a real style guide.
Comment 6 Katrin Fischer 2023-09-10 12:02:00 UTC
Verified all the buttons are now btn-primary or appear in the new staff redesign 'yellow' for the main action.

I agree with Owen about the icons for form submits.