Skip to content
OverviewSolutions
4
Comments
4
Priyanka
@priyankalad

All comments

  • Priyanka•150
    @priyankalad

    @Borub-ar This is an awsome solution.. Looking at your code, I realized many mistakes I did in my solution. Just a suggestion, you can make use of font awesome icons in this solution

  • Priyanka•150
    @priyankalad

    Hey,

    I have gone through your code, and it's pretty much cool except few alignments. I would recommend to implement this challenge using flexbox.

    Also, I noticed that feedback card containers are currently left aligned in the mobile view. To make them middle aligned, you can add 'margin: 20px auto' to the class "cards-container".

    Let me know if you need any further help

    Happy Coding :)

  • Priyanka•150
    @priyankalad

    Hi, I had earlier used background-position property with 2 values to align those 2 background images (like below code) and the struggle was real :)

    I refer one of the users code and she had used background-position property with 4 values like below in the body part

    background-position: right 45vw bottom 45vh, left 45vw top 45vh;

    [For more details on background-position] (https://css-tricks.com/almanac/properties/b/background-position/)

  • Priyanka•150
    @priyankalad

    You can apply margin-top:25vh to the class whole-container

    Alternatively, you can use flexbox to align items in your html code.