Bug 7867 - Use consistent terminology for username and password
Summary: Use consistent terminology for username and password
Status: RESOLVED DUPLICATE of bug 33766
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: master
Hardware: All All
: P5 - low trivial (vote)
Assignee: Bugs List
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-02 09:10 UTC by Adrien SAURAT
Modified: 2023-08-04 19:32 UTC (History)
10 users (show)

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


Attachments
proposed patch (2.08 KB, patch)
2012-09-21 09:47 UTC, Adrien SAURAT
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 7867: "username" instead of "login" on OPAC (2.17 KB, patch)
2012-09-23 19:35 UTC, Marc Véron
Details | Diff | Splinter Review
Bug 7867 - Same text for username/login in OPAC and intranet (2.08 KB, patch)
2014-04-04 18:07 UTC, Mark Tompsett
Details | Diff | Splinter Review
Bug 7867: Staff and OPAC login areas lack consistency (8.82 KB, patch)
2014-04-05 13:48 UTC, Mark Tompsett
Details | Diff | Splinter Review
[Signed-off] Bug 7867: Staff and OPAC login areas lack consistency (8.81 KB, patch)
2014-05-09 13:53 UTC, Marc Véron
Details | Diff | Splinter Review
Bug 7867 - Further tweaks on error messages related to login (4.61 KB, patch)
2014-05-15 14:46 UTC, Mark Tompsett
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 7867 - Further tweaks on error messages related to login (4.72 KB, patch)
2014-05-17 01:29 UTC, Bernardo Gonzalez Kriegel
Details | Diff | Splinter Review
Bug 7867: Staff and OPAC login areas lack consistency (8.84 KB, patch)
2014-05-20 11:19 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 7867 - Further tweaks on error messages related to login (4.77 KB, patch)
2014-05-20 11:19 UTC, Jonathan Druart
Details | Diff | Splinter Review
[PASSED QA] Bug 7867: Staff and OPAC login areas lack consistency (8.81 KB, patch)
2014-05-23 11:12 UTC, Kyle M Hall
Details | Diff | Splinter Review
[PASSED QA] Bug 7867 - Further tweaks on error messages related to login (4.83 KB, patch)
2014-05-23 11:12 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Adrien SAURAT 2012-04-02 09:10:15 UTC
We have username/password in the intranet login page,
and login/password in the OPAC login page.

One word for this may be better.
Comment 1 Adrien SAURAT 2012-09-21 09:47:18 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2012-09-23 19:35:21 UTC Comment hidden (obsolete)
Comment 3 Paul Poulain 2012-10-22 13:23:41 UTC
QA comment: it's a tiny patch, but on OPAC, it's written "login" on opac-user.pl too, so it's inconsistent to fix it here. 

Marking failed QA.
(As it's string freeze for 3.10 today, this patch won't be in 3.10 even if a fix is provided soon)
Comment 4 Mark Tompsett 2014-04-04 18:07:46 UTC Comment hidden (obsolete)
Comment 5 Mark Tompsett 2014-04-04 18:20:21 UTC
I'm running through the Failed QA's trying to see if there is anything I can rescue. I don't understand why this failed QA though. Did the 'Click here' say 'login' before? 'Log in' as an action link or button seems fine. The label for the text box, however, should be consistent. This patch believes it should be 'Username'.
Comment 6 Katrin Fischer 2014-04-05 07:47:57 UTC
I am not sure about username, as it might look like only the username works in the field, but we also have the option to use the cardnumber for logging in. So I thought login was not bad there, but maybe someone else has a good idea?
Comment 7 Mark Tompsett 2014-04-05 13:48:42 UTC Comment hidden (obsolete)
Comment 8 Mark Tompsett 2014-04-05 13:56:43 UTC
I have attached a 3rd patch, which is a counter patch to the first two patches. That is, they change the Staff client to 'Login' and add an explanatory tooltip to the text boxes where you are expected to enter either a username or card number.

Based on Katrin Fischer's comment 6, I think my counter patch would acceptable. I do agree that login is more generic than username.

If the latter one is acceptable for QA purposes, please obsolete the first two (the one I signed off [2.17KB] and the one I added for bootstrap [2.08KB]).
Comment 9 Marc Véron 2014-05-09 13:53:32 UTC Comment hidden (obsolete)
Comment 10 Jonathan Druart 2014-05-15 12:12:30 UTC
I am wondering if some of the following occurrences of "Username" should be replaced too:

koha-tmpl/intranet-tmpl/prog/en/columns.def:<field name="borrowers.userid">Username</field>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref:                  yes: Username and Password
koha-tmpl/intranet-tmpl/prog/en/modules/installer/auth.tt:<p><label>Username:<br />
koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt:                              <li id="ERROR_login_exist">Username/password already exists.</li>
koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt:                      Username: </label>
koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt:    <li><span class="label">Username: </span>[% userid %]</li>
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-confirmation.tt:                        <p>Username: [% borrower.userid %]</p>
koha-tmpl/opac-tmpl/bootstrap/lib/bootstrap/js/bootstrap.js:            $('#login_field').attr("placeholder", "Username or Email")
koha-tmpl/opac-tmpl/prog/en/modules/opac-registration-confirmation.tt:                    Username: [% borrower.userid %]
koha-tmpl/opac-tmpl/prog/en/modules/sco/sco-main.tt:       <label for="patronlogin">Username:</label>

Could you please have a look?
Comment 11 Mark Tompsett 2014-05-15 14:22:20 UTC
(In reply to Jonathan Druart from comment #10)
> I am wondering if some of the following occurrences of "Username" should be
> replaced too:
> 
> koha-tmpl/intranet-tmpl/prog/en/columns.def:<field
> name="borrowers.userid">Username</field>
> koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref: 
> yes: Username and Password
> koha-tmpl/intranet-tmpl/prog/en/modules/installer/auth.tt:<p><label>Username:
> <br />
> koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt:          
> <li id="ERROR_login_exist">Username/password already exists.</li>
> koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt:          
> Username: </label>
> koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt:    <li><span
> class="label">Username: </span>[% userid %]</li>
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-confirmation.tt: 
> <p>Username: [% borrower.userid %]</p>
> koha-tmpl/opac-tmpl/prog/en/modules/opac-registration-confirmation.tt:      
> Username: [% borrower.userid %]

May not apply, is it really Username or is it a generic option between card number or username? I think in cases refering directly to userid, they should say username. The login allows a login by card number, so the prompt should be more generic.

Hmmm... the returned error message should be more generic too. Will patch that up.


> koha-tmpl/opac-tmpl/bootstrap/lib/bootstrap/js/bootstrap.js:           
> $('#login_field').attr("placeholder", "Username or Email")

This is like the title attribute. Though, should it be email or card number?


> koha-tmpl/opac-tmpl/prog/en/modules/sco/sco-main.tt:       <label
> for="patronlogin">Username:</label>

Self-checkout... Yes, this should be tweaked. Will patch that up.


> Could you please have a look?

I will change it back to Adrien after this, since I really don't have that much time now allocated to Koha support.
Comment 12 Katrin Fischer 2014-05-15 14:25:54 UTC
Adrien is no longer working on Koha - mayabe switch back to unassigned then.
Comment 13 Mark Tompsett 2014-05-15 14:46:37 UTC Comment hidden (obsolete)
Comment 14 Mark Tompsett 2014-05-15 15:00:39 UTC
I'll be watching to fix my patch, but I don't have time to add more. Hence the reset to default.
Comment 15 Bernardo Gonzalez Kriegel 2014-05-17 01:29:42 UTC Comment hidden (obsolete)
Comment 16 Jonathan Druart 2014-05-20 11:19:51 UTC Comment hidden (obsolete)
Comment 17 Jonathan Druart 2014-05-20 11:19:54 UTC Comment hidden (obsolete)
Comment 18 Jonathan Druart 2014-05-20 11:20:50 UTC
Looks good to me. Maybe a native speaker should pass QA on it?
Comment 19 Kyle M Hall 2014-05-23 11:12:11 UTC
Created attachment 28457 [details] [review]
[PASSED QA] Bug 7867: Staff and OPAC login areas lack consistency

This patch:
- Changes the Staff client's login area text from "Username:"
   into "Login:", since login is more generic than username.
- Adds a tool tip to the staff client's login area 'Login:'
   textbox: "Please enter your username or card number."
- Adds a tool tip to the main OPAC screen's login area 'Login:'
   textbox using the bootstrap theme:
   "Please enter your username or card number."
- Adds a tool tip to the main OPAC screen's login area 'Login:'
   textbox using the prog theme:
   "Please enter your username or card number."
- Adds a tool tip to the OPAC screen triggered by clicking the
   'Log in to your account' link in the top right using the
   bootstrap theme: "Please enter your username or card number."
- Adds a tool tip to the OPAC screen triggered by clicking the
   'Log in to your account' link in the top right using the
   bootstrap theme: "Please enter your username or card number."

NOTE: koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-auth.tt was
      modified similarly as well, though I don't think it can be
      triggered, since the modal login screen of bootstrap is
      triggered by the 'Log in to your account' link, and it no
      longer goes to page it did before. The page still exists
      by going there by pasting in the known URL.

TEST PLAN
---------
 1) Go to staff client.
 2) Log in and change the OPAC system preference opacthemes to
    'bootstrap'.
 3) Logout.
 4) Go to OPAC in a different tab.
    -- notice staff client's "Username:" differs from the
       OPAC's "Login" text.
 5) Click the 'Log in to your account' link in the top right.
    -- notice staff client's "Username:" differs from the
       OPAC's "Login" text.
 6) Go to http://YOUROPAC/cgi-bin/koha/opac-user.pl
    -- notice staff client's "Username:" differs from the
       OPAC's "Login" text.
 7) Log in to the staff client and change the OPAC system
     preference opacthemes to 'prog'.
 8) Logout
 9) In the OPAC tab, reload the page.
    -- notice staff client's "Username:" differs from the
       OPAC's "Login" text.
10) Click the icon in the top left to return to the main Koha
    screen.
    -- notice staff client's "Username:" differs from the
       OPAC's "Login" text.
11) Apply the patch
12) Repeat steps 1-10 again.
    -- The textbox associated with the login should have a
       tooltip: "Please enter your username or card number."
13) Run the koha qa test tool.

Followed test plan (without qa test tool). Login screens display as expected.
Signed-off-by: Marc Veron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 20 Kyle M Hall 2014-05-23 11:12:34 UTC
Created attachment 28458 [details] [review]
[PASSED QA] Bug 7867 - Further tweaks on error messages related to login

As per Jonathan Druart's comments, self-checkout and error
messages related to logging in were tweaked.

TEST PLAN
----------
1) Purposefully fail to log in properly with staff client.
   -- it should say username.
2) Purposefully fail to log in properly with opac.
   -- it should say username.
3) Re-run 2 with for both prog and bootstrap themes.
4) Purposefully fail to log in properly with self-checkout.
   -- you should be prompted username and error should say
      username as well.
6) apply patch.
7) Re-run the steps, but this time messages should be related
   to login and not username.
8) run koha-qa test tools.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Work as described. No errors.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 21 Kyle M Hall 2014-05-23 11:22:00 UTC
I have some mild concerns about the use of "Login" which seems to me to be very very uncommon parlance as far as web standardization goes. Every website I've checked either uses email address ( which we don't ) or "Username" It seems like it may be better to use the phrase "Username or card number" which is much more accurate and explanatory, but also far more unwieldy. It seems that "Login" is attempting to be a replacement for this phrase specifically by *not* being the string "Username".

In summary, I find this acceptable, but I think that literally having "Username or card number" would be the best solution. However, "Login" is much more terse and I think that by and large it will not cause confusion for patrons.
Comment 22 Galen Charlton 2014-05-23 13:08:54 UTC
I have a translation question: the word "login" is ambiguous in English.  It can be a verb (to log in), or a noun referring to a username or user ID.  Are there languages that use two distinct words for those concepts?  If so, that would be a reason for running with "Username or cardnumber" or the like.
Comment 23 Katrin Fischer 2014-05-23 13:15:16 UTC
The noun and the verb are certainly different words in German.
Comment 24 Katrin Fischer 2014-05-23 13:16:04 UTC
Well, at least n this case :)
Comment 25 Mark Tompsett 2014-05-23 13:32:31 UTC
(In reply to Kyle M Hall from comment #21)
> In summary, I find this acceptable, but I think that literally having
> "Username or card number" would be the best solution. However, "Login" is
> much more terse and I think that by and large it will not cause confusion
> for patrons.

This is why you will note I added the following:
    title="Please enter your username or card number."
into the code. A hover over should clarify what is expected.
Comment 26 Galen Charlton 2014-05-23 14:00:16 UTC
(In reply to M. Tompsett from comment #25)
> (In reply to Kyle M Hall from comment #21)
> > In summary, I find this acceptable, but I think that literally having
> > "Username or card number" would be the best solution. However, "Login" is
> > much more terse and I think that by and large it will not cause confusion
> > for patrons.
> 
> This is why you will note I added the following:
>     title="Please enter your username or card number."
> into the code. A hover over should clarify what is expected.

Remember that translators will be seeing lists of words and phrases.  So if one sees the word "Login" in the translation interface... quick, tell me whether it's the noun or verb form!
Comment 27 Katrin Fischer 2014-05-23 14:09:32 UTC
Worse... it can be both in different places for an identical string.
Comment 28 Mark Tompsett 2014-05-23 14:46:13 UTC
I was thinking the other day about logging in.
Log in
Log on
Sign in
Sign on

We have lost the space which distinguishes the "Login" (noun) from "Log in" (action/verb). This loss of the space is more apparent when we talk about logging in (verb) using your login (noun).

What we have here is the mess of language. :(

What is the best way to solve this case?
Comment 29 Galen Charlton 2014-05-26 00:53:26 UTC
(In reply to M. Tompsett from comment #28)
> What is the best way to solve this case?

I still think Kyle's suggestion: "Username or card number"
Comment 30 paxed 2014-05-26 04:23:30 UTC
If only we supported the message context strings (msgctxt) ...
Comment 31 Mark Tompsett 2014-05-26 11:55:44 UTC
(In reply to paxed from comment #30)
> If only we supported the message context strings (msgctxt) ...

Just looked this up. Yes, yes this would be a better solution.
Comment 32 Hayley Pelham 2019-04-11 03:21:23 UTC
I started to fix this patch, then realised that tooltips don't display when hovering over the Login text in the OPAC, so the patch doesn't really work anymore. I am keen to standardise though, does anyone have any updated opinions on the solution for this?
Comment 33 Jonathan Druart 2020-12-16 13:57:28 UTC
(In reply to Mark Tompsett from comment #31)
> (In reply to paxed from comment #30)
> > If only we supported the message context strings (msgctxt) ...
> 
> Just looked this up. Yes, yes this would be a better solution.

We now support it.

What is the best solution to fix this problem?

(In reply to Galen Charlton from comment #29)
> (In reply to M. Tompsett from comment #28)
> > What is the best way to solve this case?
> 
> I still think Kyle's suggestion: "Username or card number"

Won't it be too long in some languages?
Comment 34 Katrin Fischer 2023-08-04 19:32:43 UTC

*** This bug has been marked as a duplicate of bug 33766 ***