Skip to content
OverviewSolutions
3
Comments
5

Ayush Sharma

@ayushsh841Bangalore,Karnataka,India85 points

Asp.Net Developer, learning React and free time gamer.

Latest solutions

Latest comments

  • Ayush Sharma•85
    @ayushsh841

    I would suggest using relative units when writing CSS so that they always adjust to the screen space like rem/%.

    Marked as helpful
  • Ayush Sharma•85
    @ayushsh841

    Hey Sarkar,

    You could align the content to the centre as on higher resolutions the component sticks to the top. Also would suggest using another div to wrap the contents of main so that it is easy to manage the content.

    Marked as helpful
  • Ayush Sharma•85
    @ayushsh841

    Hey Joel, The luxury section is not aligned with the other 2 sections. Also, you could add an empty alt="" to the images, the reason we have it has empty is because they are just icons that necessarily don't provide any extra information. This would resolve some HTML issues. You forgot to add the background colour.

    Marked as helpful
  • Ayush Sharma•85
    @ayushsh841

    For responsive design check out this channel https://www.youtube.com/channel/UCJZv4d5rbIKd4QHMPkcABCw

    Also start mobile design first so it would then be easy to scale the design to higher resolutions.

  • Ayush Sharma•85
    @ayushsh841

    Hi Shahin,

    Thanks for the feedback, will update the code based on it.