to get started I just copied in the example CSS. David Rothstein did a number on it for Gardens, and to be honest, it just looks a lot better and with seven than the default. I merged the two and added it to the main project.

The CSS is still not stellar, but much better.

Screenshots coming soon.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JacobSingh’s picture

Status: Active » Needs review
FileSize
40.72 KB
42.22 KB
David_Rothstein’s picture

It looks good to me. The main way I review CSS, though, is (a) see if the end result looks good, and then (b) glance through the file to see if anything looks extremely offensive :)

Based on that, this is good. I remember the tricky thing being to make sure the "Start upload" button behaved correctly in various scenarios - e.g., I think if you hadn't uploaded a file yet it was supposed to be grayed out. Then once you uploaded something it was blue but with styling on hover. Hard to tell from the screenshot if it is doing that.

JacobSingh’s picture

Status: Needs review » Fixed

The greying out still works, so committed! Thanks :)

Status: Fixed » Closed (fixed)

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

  • Commit ad96fd5 on master, 7.x-2.x, 8.x-1.x by JacobSingh:
    #913778 by David_Rothstein: Made plupload not look so bad