All comments

• 

  Isaac yanez• 90

  @iyanez2314

  Submitted about 1 year ago

  

  Responsive Landing Page With Flex Box

  1

  1

  0

  

  Fernando Valdés García• 415

  @fvaldes0109

  Posted about 1 year ago

  Hi, nice work! Just some small suggestions. Try adding min-height: 100vh to your body styles in order to vertically center the card. Also the elements with an active state (the button), should have a cursor: pointer. And finally, if you want to get both prices centered along the x axis, try adding: .perfume-price { display: flex; align-items: center; }

  1

  This comment has received 1 replyShow
• 

  Fernando Valdés García• 415

  @fvaldes0109

  Submitted over 2 years ago

  Any suggestion is welcome

  

  Responsive solution with basic flexbox

  3

  1

  0

  

  Fernando Valdés García• 415

  @fvaldes0109

  Posted over 2 years ago

  Thanks! That's a really nice way of centering the card

  0
• 

  Ace953• 60

  @Ace953

  Submitted over 2 years ago

  Hello everyone! Please give me your feedback and let me know if you would make any change. Thank you :)

  

  social proof section

  2

  1

  0

  

  Fernando Valdés García• 415

  @fvaldes0109

  Posted over 2 years ago

  You should remove the overflow: hidden property from the body, since on smaller screens the bottom part of the page is being cut off, and mobile views this is a bigger issue

  Marked as helpful

  1

  This comment has received 2 repliesShow
• 

  Óscar Rabadán Martínez• 150

  @Oscarandio

  Submitted over 2 years ago

  I feel very proud of this one. The only thing I am not able to figure out is how I can set the background exactly the same as the preview. I guess I need to repeat the picture and put it in the opposite side, mirroring or something. I am not sure, so any advice about it is welcome! Thanks

  

  Basic HTML and CSS 4

  1

  1

  0

  

  Fernando Valdés García• 415

  @fvaldes0109

  Posted over 2 years ago

  There are two background photos at the resources, for top and bottom. You can try creating two img elements. Set bg-pattern-top.svg as the src of the first img, and bg-pattern-bottom.svg as the src of the second one. Also both imgs must be set with position: absolute, so you can move them freely and they won't mess with the card. With that you can just experiment moving around each image with the top, left, right and bottom properties until you get the design you want.

  1

  This comment has received 1 replyShow
• 

  Fernanda Meza• 10

  @ferhammaren

  Submitted over 2 years ago

  I would like some tips on how to keep the design looking proportional when doing larger resolutions, as anything above w1440 looks too spread out and i'm not sure if that's the best solution. Any comments or things i can change or make better would be greatly appreciated!

  

  Order Summary using TailwindCSS

  2

  1

  0

  

  Fernando Valdés García• 415

  @fvaldes0109

  Posted over 2 years ago

  Try readjusting the "music icon / anual plan / change" part. Something like set its container centered and with a smaller fixed width. Also try to float: left the icon and the text, and float: right the 'change'. Hope it helps

  0