Attachment 157988 Details for Bug 33809 – Bug 33809: (follow-up) Grammar correction

[patch] Bug 33809: (follow-up) Grammar correction

Bug-33809-follow-up-Grammar-correction.patch (text/plain), 5.30 KB, created by Kyle M Hall (khall) on 2023-10-27 15:31:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2023-10-27 15:31:03 UTC

Size: 5.30 KB

patch

obsolete

>From 9a3eecfcf79fd23ac695b036617f274fb6d1b7fd Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 1 Jun 2023 12:46:22 +0000
>Subject: [PATCH] Bug 33809: (follow-up) Grammar correction
>
>This is somewhat nitpicky but I think "Show fewer..." is more correct
>than "Show less..." for strings like "Show fewer authors."
>
>'Generally, fewer is used when the number of things is counted ("fewer
>problems") whereas less is used when the number is measured ("less
>trouble" or "less time").'
>-- https://www.merriam-webster.com/words-at-play/fewer-vs-less
>
>It's not a strict rule but to my ear it sounds more correct. The brief
>"Show less" displayed on screen seems fine because the object feels
>indeterminate: "Show less information."
>
>Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> .../bootstrap/en/includes/opac-facets.inc | 22 +++++++++----------
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-facets.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-facets.inc
>index 98662ffd056..304d9f7d4fd 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-facets.inc
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-facets.inc
>@@ -101,19 +101,19 @@
> [% IF facets_loo.type_label_Location %]<a href="#" aria-label="Show more locations">Show more</a>[% END %]
> </li>
> <li class="moretoggle" style="display:none">
>- [% IF facets_loo.type_label_Authors %]<a href="#" aria-label="Show less authors">Show less</a>[% END %]
>- [% IF facets_loo.type_label_Titles %]<a href="#" aria-label="Show less titles">Show less</a>[% END %]
>- [% IF facets_loo.type_label_Topics %]<a href="#" aria-label="Show less topics">Show less</a>[% END %]
>- [% IF facets_loo.type_label_Places %]<a href="#" aria-label="Show less places">Show less</a>[% END %]
>- [% IF facets_loo.type_label_Series %]<a href="#" aria-label="Show less series">Show less</a>[% END %]
>- [% IF facets_loo.type_label_ItemTypes %]<a href="#" aria-label="Show less item types">Show less</a>[% END %]
>- [% IF facets_loo.type_label_CollectionCodes %]<a href="#" aria-label="Show less collection codes">Show less</a>[% END %]
>- [% IF facets_loo.type_label_Language %]<a href="#" aria-label="Show less languages">Show less</a>[% END %]
>+ [% IF facets_loo.type_label_Authors %]<a href="#" aria-label="Show fewer authors">Show less</a>[% END %]
>+ [% IF facets_loo.type_label_Titles %]<a href="#" aria-label="Show fewer titles">Show less</a>[% END %]
>+ [% IF facets_loo.type_label_Topics %]<a href="#" aria-label="Show fewer topics">Show less</a>[% END %]
>+ [% IF facets_loo.type_label_Places %]<a href="#" aria-label="Show fewer places">Show less</a>[% END %]
>+ [% IF facets_loo.type_label_Series %]<a href="#" aria-label="Show fewer series">Show less</a>[% END %]
>+ [% IF facets_loo.type_label_ItemTypes %]<a href="#" aria-label="Show fewer item types">Show less</a>[% END %]
>+ [% IF facets_loo.type_label_CollectionCodes %]<a href="#" aria-label="Show fewer collection codes">Show less</a>[% END %]
>+ [% IF facets_loo.type_label_Language %]<a href="#" aria-label="Show fewer languages">Show less</a>[% END %]
> [% UNLESS singleBranchMode %]
>- [% IF ( facets_loo.type_label_HomeLibrary ) %]<a href="#" aria-label="Show less home libraries">Show less</a>[% END %]
>- [% IF ( facets_loo.type_label_HoldingLibrary ) %]<a href="#" aria-label="Show less holding libraries">Show less</a>[% END %]
>+ [% IF ( facets_loo.type_label_HomeLibrary ) %]<a href="#" aria-label="Show fewer home libraries">Show less</a>[% END %]
>+ [% IF ( facets_loo.type_label_HoldingLibrary ) %]<a href="#" aria-label="Show fewer holding libraries">Show less</a>[% END %]
> [% END %]
>- [% IF facets_loo.type_label_Location %]<a href="#" aria-label="Show less locations">Show less</a>[% END %]
>+ [% IF facets_loo.type_label_Location %]<a href="#" aria-label="Show fewer locations">Show less</a>[% END %]
> </li>
> [% END %]
> </ul>
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 33809: 151568 | 151569 | 151576 | 151606 | 151607 | 151925 | 152050 | 153121 | 157986 | 157987 | 157988 | 157989