-
Notifications
You must be signed in to change notification settings - Fork 479
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
Dance page - style Amazon logo section #25950
Changes from 1 commit
bf654f2
17d3eb5
6ee38a4
773ba6a
02abe4c
2297226
f571f44
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,3 +15,13 @@ | |
margin-left: 18%; | ||
margin-right: 18%; | ||
} | ||
|
||
@media screen and (min-width: 0px) and (max-width: 512px) { | ||
.sponsor-logo-with-text { | ||
flex-direction: column; | ||
} | ||
|
||
.sponsor-logo-with-text .sponsor-logo-image { | ||
margin-top: 10px; | ||
} | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nit: Missing new line at the end of the file |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,8 +3,8 @@ | |
%div.sponsor-logo-with-text{style: "font-size: 24px; text-align: center; margin-top: 60px"} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nit: I like to keep my HAML as clean as possible. If there is an id or class, you can remove the %div part. Also, these in-line styles can be moved to your css file. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same for line 6 and throughout the file. |
||
= I18n.t(:hoc2018_dance_amazon_sponsor_section_with_logo_title) | ||
| ||
%div | ||
%div.sponsor-logo-image | ||
%img{src:"/images/fit-250/afe-dark-orange-logo.png"} | ||
|
||
%div.sponsor-subtext-description | ||
= I18n.t(:hoc2018_dance_amazon_sponsor_section_with_logo_description_with_link, amazon_future_engineer_link: 'https://www.amazonfutureengineer.com' ) | ||
= I18n.t(:hoc2018_dance_amazon_sponsor_section_with_logo_description_with_link, amazon_future_engineer_link: 'https://www.amazonfutureengineer.com') |
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.
I don't think you need to have a separate css file just for the logo classes. I'd put this inside the dance.css file which should have all the styles needed for /dance.
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.
done!