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

Katrin Fischer <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Holds Awaiting Pickup table |Holds awaiting pickup table
                   |should include the patron   |should include the patron
                   |main contact method         |main contact method
             Status|Passed QA                   |Failed QA

--- Comment #13 from Katrin Fischer <[email protected]> ---
Noting a small thing:

+                            [% CASE 'fax' %]
+                                <span>Fax</span>
+                            [% END %]
+                        </div>
+                    [% ELSE %]
+                        <div class="main_contact_method none">Main contact
method: None</div>
+                    [% END %]

I am not sure if we need the None case here. It takes up a lot of space in the
table and could also be deducted from the info now shown. We also lose the
distinction between label and content as it's now both combined into a string -
not sure if that means something for accessibility.

What do you think?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://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