Skip to content
OverviewSolutions
4
Comments
5
Ines Riahi
@inesriahi

All comments

  • Ines Riahi•110
    @inesriahiPosted almost 3 years ago

    Hi, This is great!

    I have noticed that you are setting a border around the avatar. The border is set only when the link is hovered and in small screen sizes. You may try to solve this.

    Bests

    0
  • Ines Riahi•110
    @inesriahiPosted almost 3 years ago

    Great Work!

    I have just noticed that the Learn More buttons change their position when resizing. Maybe it's a good idea if you solve this.

    Another thing is that the button's font family is the same as the paragraph. It seems that you didn't upload the files to the repository on Github.

    Regards,

    Marked as helpful
    0
  • TARUN THAKUR•30
    @bihari123Submitted almost 3 years ago

    there is something wrong with the main-background which i made. Can someone help with that?

    Used css flexbox

    4
    Ines Riahi•110
    @inesriahiPosted almost 3 years ago

    Hi, Set background-repeat: no repeat; to avoid the repetition, and add a background too using this: background: hsl(var(--pale-blue)) url("images/pattern-background-desktop.svg");

    All the Best !

    2
  • Ines Riahi•110
    @inesriahiPosted about 3 years ago

    Great Work! There is a style guiding file attached to the downloaded files. You can use it to set the colors and font as given.

    Marked as helpful
    1
  • aiswarya•30
    @AisnathanSubmitted about 3 years ago

    In mobile view, I am not able to add margin in the right and left. If I add it tends to reduce the width of the main-container. Not sure what I am missing.

    order-summary-component-main

    1
    Ines Riahi•110
    @inesriahiPosted about 3 years ago

    Hi @Aisnathan, Great Work! Maybe the reason is that you haven't set 'box-sizing: border-box' to all the elements in the page

    0