Maybe it's not a good idea to store colorbox attributes in code. For example I can use together CAPTCHA module. And it's making little mess with value like that in user_pass:
'height' => '150')

Thanks.

colorbox.module 375-382 lines

if (variable_get('colorbox_login_links', 0) == 2) {
            if (variable_get('user_register', USER_REGISTER_VISITORS_ADMINISTRATIVE_APPROVAL)) {
              $items[] = l(t('Create new account'), 'colorbox/form/user_register_form', array('query' => array('width' => '300', 'height' => 'auto'), 'attributes' => array('title' => t('Create a new user account.'), 'class' => 'colorbox-load')));
            }
            $items[] = l(t('Request new password'), 'colorbox/form/user_pass', array('query' => array('width' => '300', 'height' => '150'), 'attributes' => array('title' => t('Request new password via e-mail.'), 'class' => 'colorbox-load')));
          }
          $form['links'] = array('#markup' => theme('item_list', array('items' => $items)));
        }

Comments

frjo’s picture

Category: bug » task
frjo’s picture

Status: Active » Closed (won't fix)

The attempt to support opening various forms in a Colorbox was in hindsight never a good idea. Form error handling and form redirects are complicated, a lot of code would be needed to do it correctly. The 7.x-2.x branch removes the form support from the Colorbox module.