iwyg / filemanager

Filemanager for Symphony CMS
13 stars 11 forks source link

ui update #29

Open dtan opened 11 years ago

dtan commented 11 years ago

this is a minor change, but I applied the "symphony red" to the hover state of the "removing asset" button. I think this will give a better visual indicator similar to when you're about to delete something in the symphony ui. I also updated the p.error color to the same

iwyg commented 11 years ago

can you put this directly into the .less file? Otherwise it will be overwritten for the next build.

dtan commented 11 years ago

sure. I tried running the dev/build.sh file after making my changes in the .less file, but it didn't seem to compile it correctly. Is there something i'm missing to make the .less files build correctly? Just to run this by you as well, i changed three spots:

  1. Added @symphony_red: #dd4422;
  2. Changed the .filemanager .btn &:hover background color from darken(@bg_grey, 8%) to @symphony_red
  3. Changed p.error color of red to @symphony_red

I mean, it's pretty straight forward, but I feel like i'm missing something with the build process.

iwyg commented 11 years ago

oh, yes, my bad. The less files are not compiled within the build script. That's just r.js. You'll have to compile them manually. You can just submit the less file, and I'll compile it during the next build.

dtan commented 11 years ago

ok, i updated the less file but did not test with a build since i don't have less set up right now