@Jordananibe
Latest comments
- @Youssef5107
Hello, Jordananibe
Congratulations on completing "Qr code components" challenge . I have some advices to help you improving it , add these styles on the body: display:flex; (to use flex properties) height: 100vh; (to make sure that the body take the whole page height) align-items: centre (to center vertically ) Justify-content:center; (to center horizontally) margin is not a good way to centre your div as it fix its position so you can delete the style{ margin-left: 38%; margin-right: 50%; margin-top: 200px; } from your "container div and use better the flex properties
- @Efeh2@Youssef5107
Hello, Efeh2
Congratulations on completing this challenge . I have some advices to help you improving it , add on your "flexbox-container" div the style property (height: 100vh) that to let the div take thewhole page height so the (align-items: center) could have the vertical area that will center the div in, then delete (padding: 4rem ) as it is unneeded after adding the (height: 100vh)
- @Durgesh3829@Youssef5107
Hello, Durgesh3829
Congratulations on completing "QR code" challenge . I have some advices to help you improving it , add a new div that contains the "box" div then add: display:flex; (to use flex properties) height: 100vh; (to make sure that the new div take the whole page height) align-items: centre (to center vertically ) Justify-content:center; (to center horizontally) margin-top is not the best way to centre your div as it fix its position so you can delete the style(margin-top: 100px ) from your "box" div and use better the flex properties
- @nimishaparmar1304@Youssef5107
Hello, nimishaparmar1304
Congratulations on completing "profile-card-component-main" challenge . I have some advices to help you improving it , add a new div that contains the "qr-component" div then add: display:flex; (to use flex properties) height: 100vh; (to make sure that the new div take the whole page height) align-items: centre (to center vertically ) Justify-content:center; (to center horizontally) margin-top is not a good way to centre your div as it fix its position so you can delete the style(margin: 100px 0px 0px 800px;) from your "qr-component" div and use better the flex properties
- @maleyka13@Youssef5107
Hello, Maleyka
Congratulations on completing "ntf card component" challenge . I have some advices to help you improving it , add: display:flex;(to use flex properties) justify-content:center;(to center horizontally ) align-items: centre (to center vertically ) height: 100vh;(to make sure that the body take the whole page height) on the body to center the card, margin is not the best way to centre your div so you can delete the style(margin:100px auto) from your card div and use better the flex properties as it is more efficient on all screen sizes
- @EdiNahui@Youssef5107
Hello, EdiNahui
Congratulations on completing this challenge . I have some advices to help you improving it , add: display: flex;(to use the flex properties ) justify-content: center;(to center horizontally) align-items: center; (to center vertically ) height: 100vh;(to make sure that the body take the whole page height) on the body to center the card, when adding justify-content:center you can delete the (margin-left:auto; , margin-right:auto;) css properties from the main tag as justify-content:center do the same task more efficiently .
Marked as helpful
