Skip to content
OverviewSolutions
26
Comments
5
Fernando Valdés García
@fvaldes0109

All comments

  • Fernando Valdés García•415
    @fvaldes0109Posted over 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
  • Fernando Valdés García•415
    @fvaldes0109Posted about 3 years ago

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

    0
  • Ace953•60
    @Ace953Submitted about 3 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
    Fernando Valdés García•415
    @fvaldes0109Posted about 3 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
  • Óscar Rabadán Martínez•150
    @OscarandioSubmitted about 3 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
    Fernando Valdés García•415
    @fvaldes0109Posted about 3 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
  • Fernanda Meza•10
    @ferhammarenSubmitted about 3 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
    Fernando Valdés García•415
    @fvaldes0109Posted about 3 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