Design comparison
Solution retrospective
Please let me know your feedback.
Community feedback
- @denieldenPosted almost 3 years ago
Hi Ejerson, great work !
To center the cards on the page try using Flexbox, it can help you better: give the flexbox and
height
properties to thebody
and removemargin
from the.row
class.With bootstrap you can use the default classes:
d-flex justify-content-center align-items-center
Note: Flexbox aligns to the size of the parent container.
You can use the
vh
measurement for the height... Viewport Height handles the sizing of an element in relation to the height of the browser window.Hope this help ;)
Marked as helpful0@ejbelosoPosted almost 3 years ago@denielden thanks a lot for your feedback. This will really help to keep on coding. This was my very first time creating this kind of project on my own. Really appreciate it.
1 - @darryncodesPosted almost 3 years ago
Hi Ejerson,
Great effort on this, well done!
I noticed your icon's aren't showing - it's an easy fix, just drop the '/':
<img src="icon-luxury.svg" alt="laxury-icon" class="laxury-img">
This might be a helpful guideI also think using
flex-wrap: wrap;
should be replaced withflex-direction: column;
with a media query to transition from a row to a column for mobile. Otherwise it creates a peculiar layout at tablet. You'll need to switch the location of the border-radius with media query too!I hope that helps!
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord