Skip to content
OverviewSolutions
5
Comments
4
Rabia Ismail
@Rabiaismail21

All comments

  • Rabia Ismail•130
    @Rabiaismail21Posted over 1 year ago

    Hello BAHAA ELDEEN MOHAMED! Your submitted solution for stats preview card with animation is really amazing and it is fully reponsive as well. Great Work!

    Marked as helpful
    1
  • Rabia Ismail•130
    @Rabiaismail21Posted almost 2 years ago

    hello, fakunle! You did a good job but I have suggestion for you, Give width of 50% to both image and text content

    0
  • Rabia Ismail•130
    @Rabiaismail21Posted almost 2 years ago

    Hello Hammzyy! You tried well but some modification is needed. -instead of div use <main> tag and give a class name 'card' to it. -Create two divs inside <main>tag -Put the image in the first div and give it a width of 100% to make it responsive and fit -Put the text in another div -Give equal padding to all four sides of the <main> tag

    Marked as helpful
    0
  • Rabia Ismail•130
    @Rabiaismail21Posted almost 2 years ago

    Hello Rakesh! I have viewed your code and it needs some corrections.

    1. Inline styling is not recommended way, put all CSS code in a separate file
    2. There is no need to use the h2 and p tags multiple times one h2 tag for the title and one p tag for the subtext is enough. Use text-align: center to get the same result as the challenge project and the text will not overflow.
    3. instead of giving padding to the body, use flexbox to align your div center both in the y and x-axis
    0