Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, media-provider-css.patch, failed testing.

effulgentsia’s picture

Status: Needs work » Fixed

Whoa. We have testbot running on this project! When did that happen? I don't think the failure is related to this patch though. I committed the patch, as it's a no-brainer.

Status: Fixed » Closed (fixed)

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

Jon Pugh’s picture

Assigned: Unassigned » Jon Pugh
Status: Closed (fixed) » Needs review
FileSize
575 bytes

Those icons drive me batty. Why are they even sprites, anyway? They aren't linked...

Here's a lame patch that crops the images with CSS and goes ahead and adds :hover positioning for these non-links.

If we don't want this, we should just change those images to not be sprites?

gmclelland’s picture

Why don't we get rid of the big provider icons all together and just use fav icons for each provider? We can also show each provider using a vertical accordion. When you click on a provider name it would expand to show the usage examples.

See #1494416: UX - Providers - List Usage Examples

Devin Carlson’s picture

Dave Reid’s picture

I'd rather just use text for the providers at this point. This has been horribly broken for so long.

Devin Carlson’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev
Assigned: Jon Pugh » Devin Carlson
FileSize
1.81 KB

A patch to convert the display of providers into an unordered list of provider titles per #7.

I haven't removed 'image' from media_internet.api.php but that could be done if required.

Dave Reid’s picture

Here's the code I had locally with a little bit cleaner output and removes the CSS from media.css. If we want to re-add it it should be going in media_internet.css and not the base CSS.

Dave Reid’s picture

With the CSS removal.

Dave Reid’s picture

With a nice class for each list item so that it can be styled by individual modules.

Dave Reid’s picture

Here's how it looks with the patch in #11.

Add file | drupal7dev.local (mysql).png

Dave Reid’s picture

This would also make it possible to do something like this:

.media-internet-providers li.youtube {
  list-style-image: url(/sites/all/modules/media_youtube/images/youtube_16.png);
}

.media-internet-providers li.vimeo {
  list-style-image: url(/sites/all/modules/media_vimeo/images/vimeo_16.png);
}

which would look like:
Add file | drupal7dev.local (mysql).png

Dave Reid’s picture

Status: Fixed » Closed (fixed)

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