-
Notifications
You must be signed in to change notification settings - Fork 5
Adressing petres comments for desktop (responsive hero is still pending) #29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: Christian Bromann <[email protected]>
christian-bromann
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
Yeah, taking the social icons out broke it. |
|
@christian-bromann |
christian-bromann
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
…ng) (#29) * Removing unused images * Using vector images for logos * Removing social icons on hero * More space between title and heading * Moving values to its own partial * Adjusting box shadow * Adjusting font color * Using vectors to avoid fuzzy look * Fixing links and button look * Fixing link styling for events * Fixing blog text color and link * Left align copyright * Using text links for projects * Adjusting blog summary shadow * Fixing styles for docs * Adjusting spacing, overwrite template * Update static/sass/components/buttons.scss Co-authored-by: Christian Bromann <[email protected]> * Leaving social to avoid breaking the header Co-authored-by: Christian Bromann <[email protected]> e4ba5b5


Home page changes
Project page changes
Docs page changes