It looks like multiselect lists are inheriting styles from the standard select list and look ridiculous:

Create_test___Century21_Roy_B._Hull.png

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrfelton’s picture

Yes, this actually makes the selector pretty much impossible to use. I can't scroll down to see the items lower down. I hack fixed this in our own profile by forcing it to only allow single selections:

/**
 * Implements hook_views_default_view_alter().
 */
function alumni_base_views_default_views_alter(&$views) {
  if (isset($views['commerce_backoffice_content'])) {
    // Only allow single selection on the content admin screen.
    // Temporary fix for https://drupal.org/node/2054091
    $handler =& $views['commerce_backoffice_content']->display['default']->handler;
    $handler->display->display_options['filters']['non_product_display_node_type']['expose']['multiple'] = FALSE;
  }
}
dudenhofer’s picture

I can see how this would be confusing... not sure how it makes it impossible to use :P Nothing is happening to the selector except adding a background image. We can remove the background image and some padding for the multiple select, but I'm not sure how that will fix the issue in comment #1.

torpy’s picture

The visual issue is happening for me too but I can't reproduce the issue in #1. It might be due to a different problem.

bleen’s picture

the usability issue in #1 might be localized to OSX which hides scrollbars

alexweber’s picture

I've been running with this override locally:

// Fix multiple select boxes.
select.form-select {

  &[multiple="multiple"] {
    background: #fff;
    padding: 5px;
  }

}

If I can find some time I'll roll a patch... :)

Note: it doesn't fix the OSX + Chrome quirks with scrolling in multiple selects, thats not Shiny's bug IMO. It just removes the background and padding for multiple selects so they look normal again

alexweber’s picture

Status: Active » Needs review
FileSize
426 bytes

+1 for not procrastinating :)

bleen’s picture

Status: Needs review » Reviewed & tested by the community

RTBC

dudenhofer’s picture

Status: Reviewed & tested by the community » Fixed

applied and compiled the css. Thanks!

alexweber’s picture

Awesome, thanks!

PS - Git commit credits go a long way ;)

dudenhofer’s picture

Sorry, I didn't realize I had committed that wrong. See http://drupalcode.org/project/shiny.git/commit/f7815a9

alexweber’s picture

Awesome thanks dude! :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.