-
Notifications
You must be signed in to change notification settings - Fork 9
General thoughts for design tweaks #109
Comments
Just my thoughts:
On 3 August 2013 13:25, Mia notifications@github.com wrote:
Scott Kleinman |
I will offer some suggestions when I'm at a computer.
|
|
A few other issues I'd like to get to: • I made a class called ".main" that, if added to the about and connect page divs would update the styling of the h2s + h3s to the same red color found on the homepage, as well as the Typekit webfonts. The class rule is in the "base.css" folder but the class isn't assigned in the HTML. • increase the margins on the left / right of the about + connect page. As of right now, the line length is a bit long for optimal reading but this is quasi-minor. 30px on the left (to better align with the serendipomatic logo) and 10px on the right should do it. • the hover state on the "Make Some Magic" button should have a shadow of #FFFFFF instead of #000000. The black shadow and black text look weird. • on the results page, the "Go Home" should definitely have more separation from the "Save Results" action and perhaps that entire section can move down a bit. I'm still a bit unclear where I am supposed to make any of these changes and terrified of screwing things up. |
@fontnerd - I'll have to check back through the discussion but if you push changes from your laptop or directly on the site, they should be honoured as people will check for any more recent changes (with re-merge) and merge any lines individually before committing their own stuff. Can we also have external links as a slightly different colour (perhaps darker so less prominent) than internal app links? The idea being that it helps keep attention on the main call-to-action in using the site (especially the first time). |
Forgot to add, when you submit to master it should deploy to http://dev-serendipomatic.herokuapp.com/ so you can see it all... (Clearly I should either edit or respond to OWOT emails, not both!) |
The text was updated successfully, but these errors were encountered: