-
Notifications
You must be signed in to change notification settings - Fork 1.4k
adjusted icons size #5613
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
adjusted icons size #5613
Conversation
Signed-off-by: Hargun Kaur <[email protected]>
🚀 Preview for commit 53db506 at: https://667c97532ed1181c1f160a91--layer5.netlify.app |
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.
The logos under "Layer5 Social Backgrounds" and "Five, our mascot" are too small and need to be adjusted.
@hargunkaur286 Thank you for your contribution. |
margin-left: 0px; | ||
} | ||
.ImgDiv img{ |
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.
how you are ensuring using this css that the image will set its dimensions accordingly based on screen size
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.
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.
Can you please review now? @sudhanshutech @iArchitSharma I have set the width to be auto so that it gets adjusted to different screen sizes and rather set the height.
Signed-off-by: Hargun Kaur <[email protected]>
🚀 Preview for commit 4fce6a5 at: https://668847deabe78a72ecb44df0--layer5.netlify.app |
@ashparshp what do you think? Good to go now or more feedback needed? |
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.
@leecalcote looks great and is ready to go. Thanks, @hargunkaur286, for making the changes!
Description
This PR fixes #5595
Notes for Reviewers
Signed commits