If there are no OpenID identities in user/%uid/openid page, it shows plain table header with no content inside table. According to http://drupal.org/node/604342 and #504428: Roll-out 'Empty text' pattern to other admin pages. empty table should contain an explanation and/or call to action.
Proposed wording if there is no content in the table:
No OpenID identities available.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | openid_empty_pattern-D7-1473214-6.patch | 486 bytes | roborn |
| #3 | openid_empty_pattern-D8-1473214-2.patch | 506 bytes | nmudgal |
| #1 | drupal-openid-empty-pattern-1473214-2.patch | 489 bytes | kika |
| empty_openid_user.png | 78.86 KB | kika |
Comments
Comment #1
kika commentedHere's the patch.
Comment #2
c960657 commentedI wonder if “available” is the best word to use in this context. How about 'No OpenID identities for this account'?
Comment #3
nmudgal commentedI merged both #1, #2 & here is proposed wordings:
No OpenID identities available for this account.Patch attached.
Thanks.
Comment #4
roborn commentedYes, sounds good.
The patch applies cleanly and works as expected. RTBC.
Comment #5
webchickAwesome! Thanks a lot for this nice little improvement.
Committed and pushed to 8.x. I think we need this patch in 7.x. as well?
Comment #6
roborn commentedAgree :)
Here's the patch for D7
Comment #7
devin carlson commentedThe patch in #6 applied cleanly and added the empty text to the "OpenID identities" tab on user pages.
Comment #8
webchickThis one is a little risky for D7 because it is a user-facing string, but it's a pretty technical screen for them to end up at, so I think it's ok to break translations here. If David disagrees, he should feel free to revert this patch.
Committed and pushed to 7.x. Thanks!
Comment #10
David_Rothstein commentedThis is already mentioned in CHANGELOG.txt, so adding to the 7.17 release notes as well.