Bug fixes and improvements

thoughtworks / letshelp.it

http://letshelp.it

18 stars 13 forks source link

Bug fixes and improvements #40

Closed nataliarsand closed 13 years ago

nataliarsand commented 13 years ago

Fixing bugs, improving here and there..

List of reported bugs:

better background image (that one is from AFASO right?)
organization boxes are not aligned
about page with lot of contrast (background x text) '
not sure we can keep that "Let' Help it is a project idealized by TW-Brazil"
background image: its quality is very low, makes somewhat hard to read the white text
organization list: in my FF it's displaying 2 organizations side by side. it could take benefit of higher resolutions by displaying more organizations side by side

Please, keep the suggestions coming!

lucastorri commented 13 years ago

Hey Natalia, thanks a lot. Just to let you know, I will take a better look on it tomorrow and talk with some of the guys, ok? Anyway thank you in advance again.

cv commented 13 years ago

I couldn't get this one to apply cleanly — Natalia, are you up to date? If not, could you merge in the changes from Rafael and let us know?

Cheers!

lucastorri commented 13 years ago

Wow! Really liked the new logo and colors... they look great!

I had an issue adding organizations with big names, which lead the following text to get out of the frame.

lucastorri commented 13 years ago

uhm, the about page is all in one strait column.

rmagrin commented 13 years ago

Hey Natalia...I applied your changes to the 'new_layout' branch, but there seems to be two files missing your repository: niceforms-default.css and niceforms.js.

Anyway, I commented the parts of the code where we load this files and pushed it to http://letshelpit-dev.heroku.com/. Please take a look there and verify if the merge was ok.

Thanks! You were very fast working on the suggestions!

nataliarsand commented 13 years ago

hey cv, I had Rafael changes, we were working together. Magrin, I forgot to take that code out, was testing that script and didn't like it, thanks for commenting that out, I'll delete it for the next merge.

Will take a look at the text getting out of the frame.

Thanks everyone!

rmagrin commented 13 years ago

Hey Natalia...cv was talking about the changes in the code made in the thoughtworks/letshelp.it repository. I made pull request to your repository to bring it up to date.