https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15824

--- Comment #6 from M. Tompsett <[email protected]> ---
Comment on attachment 48565
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48565
Bug 15824 - 'Done' button is unclear on batch item modification and deletion

Review of attachment 48565:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=15824&attachment=48565)
-----------------------------------------------------------------

Comments base on eyeballing.

::: koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-del.tt
@@ +161,5 @@
> +            [% ELSIF src %]
> +                <a href="[% src %]">Return to where you were</a>
> +            [% ELSE %]
> +                <a href="/cgi-bin/koha/tools/batchMod.pl?del=1">Return to 
> batch item deletion</a>
> +            [% END %]

Why not style this identical to the existing block below?

@@ +192,4 @@
>  
>      <p>
>          [% IF src == 'CATALOGUING' # from catalogue/detail.pl > Delete items 
> in a batch%]
> +           <a class="btn" 
> href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% biblionumber %]"><i 
> class="fa fa-check-square-o"></i> Done</a>

Why not change this to "Return to the record", as above?

@@ +196,2 @@
>          [% ELSIF src %]
> +           <a class="btn" href="[% src %]"><i class="fa 
> fa-check-square-o"></i> Done</a>

Why not change this to "Return to where you were", as above?

::: koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-edit.tt
@@ +66,5 @@
> +                  [% ELSIF src %]
> +                      <a href="[% src %]">Return to where you were</a>
> +                  [% ELSE %]
> +                      <a href="/cgi-bin/koha/tools/batchMod.pl">Return to 
> batch item modification</a>
> +                  [% END %]

Why not style this block as was done below?

@@ +245,4 @@
>  [% ELSE %] <!-- // show -->
>  <fieldset class="action">
>  [% IF src == 'CATALOGUING' # from catalogue/detail.pl > Edit items in a 
> batch%]
> +   <a class="btn" href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% 
> biblionumber %]"><i class="fa fa-check-square-o"></i> Done</a>

Why not change this to "Return to where you were", as above?

@@ +249,2 @@
>  [% ELSIF src %]
> +   <a class="btn" href="[% src %]"><i class="fa fa-check-square-o"></i> 
> Done</a>

Why not change this to "Return to where you were", as above?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to