Skip to content
OverviewSolutions
5
Comments
5
RaffayElahi
@RaffayElahi

All comments

  • RaffayElahi•130
    @RaffayElahiPosted over 1 year ago

    Hey @Priyesh-Singh-Waldia! I viewed your code, it was great. This was the following practice that I would recommend you to follow out:

    • Make structured HTML pages. I saw your code, it was randomly containing tags like aside and div class='container'. Although aside tags is used for ads as majority and when you will do the SEO o your website, it will have a great impact on it. I hope you will follow it, Happy coding, Regards.
    Marked as helpful
    0
  • RaffayElahi•130
    @RaffayElahiPosted over 1 year ago

    Hey @younasBnHasainar! I saw your code, it was great but some practices will greatly help you in your long-term web development journey. Those practices are:-

    • I saw that your HTML wasn't structured as you were using randomly h2 and h4 tags, try using heading tags according to the importance of heading and heading number. This practice will make your SEO friendly also use section tags for covering your whole main content instead of div. These look like small negligible mistakes but these make difference in readibility and access to your code.
    • Second, you could make the whole body a flex container in order to center out the main div instead of using out margins.
    • Try using ul and li for listing out things instead of listing out things in p tags. Also try to use flex containers as much as possible to make your content aligning out either in row or column or wither centering it or giving out space evenly. Flexbox is really helpful.

    Hope these suggestion were helpful. Happing coding and regards.

    0
  • RaffayElahi•130
    @RaffayElahiPosted over 1 year ago

    Hey @leveau10! I viewed your code, it was great code. Great work done by you. I noticed the following things:-

    • You were using margins to center out divs. Instead of that you can use justify-content:center; property to center out your content and also giving out widths to the 'div' to make sure that it doesn't overflow or take extra text.
    • Also try to use section tag and similar HTML tags to make your code readable which will greatly help you in your further web development journey.

    Hope you find these suggestions helpful.

    Happy coding and regards.

    Marked as helpful
    2
  • RaffayElahi•130
    @RaffayElahiPosted over 1 year ago

    Hey @sumyta12, The website looks phenomenal, I don't know much about the api side but it was great. I would prefer you to use tailwind css for adding and managing responsiveness. However, the website was great. Regards.

    Marked as helpful
    0
  • Harsh•305
    @1MpedenceSubmitted over 3 years ago

    New to HTML, CSS, and JS. Learned for the first time how to use range input and checkbox. It was fun :P. Would appreciate any feedback and advice. ThankYou :D

    Interactive-pricing-component

    1
    RaffayElahi•130
    @RaffayElahiPosted over 3 years ago

    There is a simple error in this website. You should have to update the pageviews as the scrollable slider is being scrolled. It is easy to update and is written in the challenge0

    0