diff options
author | Brion Vibber <brion@pobox.com> | 2010-10-20 16:14:32 -0700 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-10-20 16:14:32 -0700 |
commit | c24f4ddfdbf00f024730c1f76e3f17e8e3999d85 (patch) | |
tree | 8e86d93a26f446df9890dfa3447e32991f51f3ac /plugins/OpenID/OpenIDPlugin.php | |
parent | bcc06d05e8b0ec4ac9d91ac65e6905d02c4b43cc (diff) |
Pretty up the OpenID variant of the OAuth login form a bit; change the 'Allow' button to 'Continue' so we're not confused why we get the form again after authenticating.
Diffstat (limited to 'plugins/OpenID/OpenIDPlugin.php')
-rw-r--r-- | plugins/OpenID/OpenIDPlugin.php | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/plugins/OpenID/OpenIDPlugin.php b/plugins/OpenID/OpenIDPlugin.php index 0fa50139d..c3dbd3068 100644 --- a/plugins/OpenID/OpenIDPlugin.php +++ b/plugins/OpenID/OpenIDPlugin.php @@ -655,11 +655,13 @@ class OpenIDPlugin extends Plugin return true; } - function onStartOAuthLoginForm($action) + function onStartOAuthLoginForm($action, &$button) { if (common_config('site', 'openidonly')) { // Cancel the regular password login form, we won't need it. $this->showOAuthLoginForm($action); + // TRANS: button label for OAuth authorization page when needing OpenID authentication first. + $button = _m('BUTTON', 'Continue'); return false; } else { // Leave the regular password login form in place. @@ -674,6 +676,10 @@ class OpenIDPlugin extends Plugin */ protected function showOAuthLoginForm($action) { + $action->elementStart('fieldset'); + // TRANS: OpenID plugin logon form legend. + $action->element('legend', null, _m('OpenID login')); + $action->elementStart('ul', 'form_data'); $action->elementStart('li'); $provider = common_config('openid', 'trusted_provider'); @@ -699,6 +705,8 @@ class OpenIDPlugin extends Plugin } $action->elementEnd('li'); $action->elementEnd('ul'); + + $action->elementEnd('fieldset'); } /** |