Skip to content
OverviewSolutions
9
Comments
8

Tiago Pereira

@BuildAndBreakZurich, Switzerland200 points

Aspiring Full-Stack Web Developer 💻 I’m on a journey to master both front-end and back-end development! Passionate about building websites and web Apps 🚀

I’m currently learning...

Express JS

Latest solutions

View more solutions

Latest comments

  • Tiago Pereira•200
    @BuildAndBreakPosted 3 months ago

    Hi, you've built a solid foundation for form validation, and with a few small tweaks, it'll be even more user-friendly.

    Try to improve this:

    -Form is never submitted – The submit event calls e.preventDefault(), but there’s no condition to allow submission when all inputs are valid.

    -Validation of all fields before preventing submission: validateInput(input) should be called for all fields at the same time, displaying all errors.

    Keep up the excellent work!

    Marked as helpful
    0
  • Tiago Pereira•200
    @BuildAndBreakPosted 3 months ago

    Your HTML looks good! Just a couple of things:

    Headings: Maybe change one of the <h3> to <h1> for better structure. Footer: The <footer> tag for each item is a bit off—try using a <div> or <section> instead.

    Keep going :)

    Marked as helpful
    1
  • Tiago Pereira•200
    @BuildAndBreakPosted 3 months ago

    Nice job for completing the challenge! Looks great! Code is clean, and it’s responsive.

    These tweaks can improve structure and accessibility! Consider improve alt text to be more descriptive like: "Gabrielle Essence Eau De Parfum bottle". Group related content using tags like:

    <article>, <section>, or <figure>. Add accessibility features like aria-label to links and buttons.

    You're doing great!

    Marked as helpful
    0
  • Tiago Pereira•200
    @BuildAndBreakPosted 3 months ago

    Hey, nice job on the Interactive Rating Component! It looks smooth, and everything works well. Solid work—keep it up! 🚀

    0
  • Tiago Pereira•200
    @BuildAndBreakPosted 6 months ago

    Error 404 , there isn't a GitHub Pages site here.

    0
  • Gigi SN•40
    @jeanethsnSubmitted 6 months ago
    What are you most proud of, and what would you do differently next time?

    I'm proud of using variables for spacing, colors and font size.

    Social links profile main solution

    1
    Tiago Pereira•200
    @BuildAndBreakPosted 6 months ago

    Hi, great job completing this project, @jeanethsn!

    Your code is clean, readable, and makes effective use of semantic HTML.

    I also appreciate the use of css variables, it's a great habit that will be invaluable in larger projects, so getting comfortable with them early is definitely a smart move.

    A couple of suggestions:

    -Fixed Width on Section:

    I noticed you set a fixed width of 310px on the <section> element. While this will work on most devices, I’d recommend using a percentage (%) for more flexibility and adding a max-width of 310px if needed. This will make your layout more responsive across different screen sizes.

    -Clickable Links:

    Consider adding links within each list item to make the social media options clickable. Redirecting users to these platforms could be a nice enhancement and why not make your personal social links profile? Think about that :)

    Tip: It looks like there are two closing <main> tags by mistake, so double-check that part.

    Keep up the great work! Greetings, Tiago

    0
View more comments