Skip to content
OverviewSolutions
3
Comments
79
khushboo khatwani
@khushi0909

All comments

  • khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago
    1. Read about h1 tag its the most important and every website should have atleast one https://www.semrush.com/blog/h1-tag/
    0
  • khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago

    1)you should learn about semantics HTML and using it for ex footer ,main etc

    2)image should always have alt="some description". some description about image should always be there for accessibility reasons

    https://www.davidmacd.com/blog/alternate-text-for-css-background-images.html

    3)Read about h1 tag its the most important and every website should have atleast one https://www.semrush.com/blog/h1-tag/

    4)your text is going outside the center

    5)i liked that you used the media queries for responsiveness ,but you need to practice a little bit because even with that, in every screen size ,your text are going out of the container and its not good

    All the best

    Marked as helpful
    0
  • khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago

    1)you should learn about semantics HTML and using it for ex footer ,main etc

    2)Read about h1 tag its the most important and every website should have at least one https://www.semrush.com/blog/h1-tag/

    3)now it looks good on every screen ,but you can read about media queries for responsiveness for the future

    solution looks good ,All the best

    Marked as helpful
    0
  • RKanna•150
    @RKannaSubmitted over 1 year ago

    I tried this one, Can any one suggest me how to improve this code?

    Responsive CSS

    2
    khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago

    1)Read about h1 tag its the most important and every website should have atleast one https://www.semrush.com/blog/h1-tag/

    2)you should learn about semantics HTML and using it for ex footer ,main etc

    3)image should always have alt="some description". some description about image should always be there for accessibility reasons

    https://www.davidmacd.com/blog/alternate-text-for-css-background-images.html

    4)i feel something wrong with the positioning of the background image or color you have used ,its creating some problem in responsiveness view for different screen and you are getting lot of scroll bar both vertically and horizontally ,it should not be ideally as per design like this

    5)i personally feel using flexbox in the body tag will also be better' and mentioning in body min-height as 100vh may help (read about min-height 100vh more)

    All the best

    Marked as helpful
    0
  • khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago

    1 you should learn about semantics HTML and using it for ex footer ,main etc

    2)Read about h1 tag its the most important and every website should have atleast one https://www.semrush.com/blog/h1-tag/

    3)width: 275px; ,instead of fixed card width ,you should try to use min-width or max-width accordingly for better responsiveness

    All the best

    Marked as helpful
    1
  • khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago

    1)image should always have alt="some description". some description about image should always be there for accessibility reasons

    2)you should learn about semantics HTML and using it for ex footer ,main etc

    3)Read about h1 tag its the most important and every website should have at least one https://www.semrush.com/blog/h1-tag/

    4)now it looks good on every screen ,but you can read about media queries for responsiveness for the future

    All the best

    Marked as helpful
    0
  • khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago

    1)image should always have alt="some description". some description about image should always be there for accessibility reasons

    2)now it looks good on every screen ,but you can read about media queries for responsiveness for the future

    All the best

    1
  • Md. Sohel Rana•20
    @Sohel-Rana0Submitted over 1 year ago

    This solution so easy. If you want to more about please contact with me. Thank you .

    Qr-code-component

    3
    khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago
    1. image should always have alt="some description". some description about image should always be there for accessibility reasons

    2)container is not in the center as required by the design ,you should make it center by using flexbox or transform properties reead more about it

    3)Responsiveness has some problem ,if i check this in mobile size of 320 px qr code goes out of the container ,in this context you can also read more about the media queries

    All the best

    0
  • Igor•230
    @MIU-czSubmitted over 1 year ago

    css

    3
    khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago

    1)you should learn about semantics HTML and using it for ex footer ,main etc

    2)Read about h1 tag its the most important and every website should have atleast one https://www.semrush.com/blog/h1-tag/

    3)now it looks good on every screen ,but you can read about media queries for responsiveness for the future

    All the best

    Marked as helpful
    0
  • khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago

    1)Read about h1 tag its the most important and every website should have atleast one https://www.semrush.com/blog/h1-tag/

    2)you should learn about semantics HTML and using it for ex footer ,main etc

    3)now it looks good on every screen ,but you can read about media queries for responsiveness for the future

    All the best

    0
  • khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago
    1. you should learn about semantics HTML and using it for ex footer ,main etc

    2)Read about h1 tag its the most important and every website should have atleast one https://www.semrush.com/blog/h1-tag/

    1. width: 300px; height: 480px; you have fixed width and height for the containers ,ideally it should be max-width or min-width accordingly and it helps in better responsiveness .

    2. left: 50%; top: 50%; margin-left: -150px; margin-top: -250px;

    instead of this try transform property that will be good ,putting so much margin is not a good practice and may create the problem in future

    left: 50%; top: 50%; transform:translate(50%,50%) read more about it

    5)with mobile screen i find it too small container ,may be feels like it will not be much readable to the user

    now it looks good on every screen ,but you can read about media queries for responsiveness for the future

    All the best

    0
  • khushboo khatwani•1,070
    @khushi0909Posted over 1 year ago
    1. i find the content big as compared to the expected design

    2)Read about h1 tag its the most important and every website should have atleast one https://www.semrush.com/blog/h1-tag/

    3)now it looks good on every screen ,but you can read about media queries for responsiveness for the future

    All the best

    Marked as helpful
    0