Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • @Mageshwari-Balaguru

    Submitted

    What are you most proud of, and what would you do differently next time?

    This layout took me around 6 hours. Next time will try to reduce the time. also try to use some CSS methodologies like BEM. I just recently come to know there were some methodologies is there which will make our code more readable, understandable.

    What specific areas of your project would you like help with?

    In the flex container, .items-container the paragraph text which is in speech class was not occupy the whole size. i tried max-width to fit-content - won't work, so deleted that.

    @Adebukol

    Posted

    your code uses semantic HTML effectively with elements like <main> and <footer>. The use of <h1> tags for the primary headings in each testimonial is appropriate, emphasizing the importance of the content. Improvements: Consider replacing some of the <div> elements with more semantic tags. For example, <section> could be used for each .items-container to better indicate that each section is a standalone block of content. 2. Accessibility Strengths: The alt attributes are correctly used for images, which is crucial for screen readers and improves accessibility. Improvements: The headings structure could be improved by ensuring there’s a logical flow from <h1> to <h2> (or further down if needed) for better accessibility. For instance, having multiple <h1> tags might not be ideal; consider using <h2> for subsequent headings. 3. Responsive Layout your code appears to be structured in a way that could be easily adapted for responsiveness, especially with the container and wrapper classes.

    0
  • @Adebukol

    Posted

    Welldone for making efforts to get this done. semantic HTML followed Layout not bad Readable code Slight differences between the design and the solution. layout responsive. great job keep up the good work it can only get better

    1
  • @Adebukol

    Posted

    Welldone, however your live site does not meet up with the design. your code does not include semantic HTML, Your code is responsive on all devices, nice work there. great effort you've put into this.

    0
  • @Adebukol

    Posted

    Nice work It included semantic html. It is accessible The solution differs from the design, the layouts are way bigger and the responsiveness needs to be worked on as my mobile view of the site was not looking good at all. But in all great job.

    0
  • @Adebukol

    Posted

    well Shoaib,The social media profile webpage is visually appealing, with a clean design, intuitive navigation, this is quite impressive. keep up the good work

    Marked as helpful

    1
  • @Adebukol

    Posted

    Well done @pablo However here are a few things I can say regarding your design, it's uniquely yours but not in accordance with the design UI, I do not have access to your codes as well perhaps you used a repository name that didn't align with the design.

    0
  • @Adebukol

    Posted

    Nice trial, however your card height differs with the design. You didn't make use of enough padding, your border radius as well.

    Marked as helpful

    1
  • @llr3v0ll

    Submitted

    while doing this I encountered an issue. For the radio-buttons I couldn't get the "checked" button text color to be white therefore I made the text white by default. Any idea of how that could be solved?