And yes, I firmly believe this is a critical bug (not a beta-breaker, but needs to be fixed before release). The current text at user/#/openid is wholly inadequate:
"You may login to this site using an OpenID. You may add your OpenId URLs below, and also see a list of any OpenIDs which have already been added."
- What the heck is OpenID?
- What the heck is an OpenID URL?
- If I don't have one, where do I get it?
- Why would I want one?
- OMG! WHY DID THE PAGE JUST TURN INTO LIVEJOURNAL!? THE WORLD IS ENDING! AHHH!!!
Remember that this is *user*-facing text. We can't assume this user has any idea what OpenID is, so we need to guide them along.
Comments
Comment #1
eaton commentedOMGZORZ!
Comment #2
eaton commentedAfter a bit of discussion on IRC, I proposed this as a starting point:
Comment #3
HedgeMage commentedI showed what Eaton suggested to a couple of non-technical users. Both found "a standard way" confusing. Neither knows what technological standard is; they just of wondered what OpenID is and how it was standard if they'd never heard of it. Also, "from another site" made them think that the site this appeared on must also be an OpenID provider, and "to add OpenIDs..." was similarly confusing. How does the following sound?
Comment #4
squidly commentedI think a link to the Drupal handbook page would be good. That and also putting a link to more info about OpenID would be a good idea.
Comment #5
KentBye commented@ squidly: Re: "putting a link to more info about OpenID would be a good idea" -- there's already a link to openid.net, but I do think a couple of more links might be helpful as well.
For example, some people may already have an OpenID and not even realize it -- like for users of livejournal, wordpress.com or vox. So a link to a wiki with a list of current OpenID providers might be helpful -- here's one possible listing: http://openid.net/wiki/index.php/OpenIDServers
Another thing that might be good is to give an example syntax of an "OpenID URL." An exmple that's used here is -- For example: happygirl.bloghost.com We could change it to For example: myusername.openidprovider.com
And it might be helpful to also link to that How's it work?" page since it's a little mysterious to most people how such a thing could exist safely. It gives a pretty basic overview and also has links to wonky flowcharts like this one.
So integrating these suggestions, here's what I'd propose building off of HedgeMage's suggestion.
Not sure if that's too long, but there's definitely some more pointers and I think it flows a bit better.
Comment #6
Tresler commentedAgreed, what is there does not explain enough. Also worth looking at the existing text at /admin/help/openid though
The two should reflect each other some.
I will hammer out a patch in the next few hours. Then we have a place to work from .
Comment #7
Tresler commentedI'm begining to think that is a little to much info, and I'd also like a help page for users - not admin/help/openid as anonymous users can't access that - that explains a little of the how of openid. Basically some of the text on openid.net/about
That said, I'll work it out by the end of the day.
Comment #8
Tresler commentedSo, I wanted to give the user some sort of idea what OpenID was, and still keep it as short and simple as possible.
I also ammended thehelp page to give the administer an idea as to what was going on (OMG! Livejournal haxxed my f$*King site!!!).
For your reading pleasure, I've pasted the text of both here, patch attached.
and on the help page:
Comment #9
Tresler commentedgah. I had been playing with the wording 'trusted site' but decided to stick with OenID server to keep the branding and the true terminology. But I left one in above. This patch fixes that.
Comment #10
moshe weitzman commentedthats better.
Comment #11
gábor hojtsyHm, a few problems (although note that I am not a native English speaker):
- Sometimes there are two spaces in the middle of a sentence, and there are two between sentences, which is not right.
- concatenation at the end of t() calls with the p tag seems to violate coding standards (whitespace required before the dot)
- "When next you login" sounds like broken... Maybe "Next time you log in"
- "you will be able to use the username and login from your OpenID identity" also sounds broken. I only need to pass the URL right, and then provide a proof of ownership of that URL elsewhere, right?
- 'at their website' could be 'at their website', so more words are clickable (easier to click on the link)
- in Drupal core we are trying to use more to the point URL placeholders, instead of boring @url and @url2, so better placeholders would be nicer.
- drupal.module is not going to be included in Drupal 6, so new users will have no idea whatsoever what that note might mean (contrib will host the module under a different name). Maybe this needs to be made a bit generic, eg: "Logging in via OpenID is far more secure than the distributed login solutions which came built-in with Drupal before." instead of "Logging in via OpenID is far more secure than the previous method using drupal.module."
Comment #12
webchickAgree with everything else Gabor said, but:
Comment #13
webchickRessin' fressin... :P
Comment #14
pcwick commentedBeginning with Tresler's text I came up with the following. My knowledge of OpenID is limited, so if you like the text please check for accuracy. Sorry, I won't have a patch environment set up for several days.
Comment #15
pcwick commentedI was inconsistent with "website" and "web site". Which is preferred?
Comment #16
alpritt commentedThis is an amendment to Tresler's patch from comment #9.
re: comment #11:
- white spacing fixed
- Broken language, reworked.
- I've changed 'at their website' to 'this website' because I felt 'their' suggested that OpenID was owned by a company or organisation.
- I've changed the URL placeholders. Have I done this correctly?
- I agree with webchick's comments in comment #12 and have removed the reference to drupal.module.
- I've added the 'myusername.openidprovider.com' example from comment #5.
I've also added a little to the admin/help/openid page pointing out that this isn't a trust system and so we need email verification still. And I have reworked the rest of that text, hopefully to make the examples clearer (but it is late so it is quite possible I've not made it worse ;) ).
Finally this is my first patch ever, so please tell me if I've done anything wrong. :)
Comment #17
alpritt commentedWhoops, I was rude and ignored pcwick. This patch draws from his reworking a little and rearranges some of the text into a more logical flow on the user/%/openid page.
Comment #18
alpritt commentedSorry, I put whitespace errors in the last patch. Tiredness.
Comment #19
dries commentedThanks for all the hard work folks. I've committed alpritt's version of the patch. If you find more issues, we can tackle those in follow-up patches.
Comment #20
(not verified) commented