Skip to content
OverviewSolutions
5
Comments
4
Emmanuel-Xs
@Emmanuel-Xs

All comments

  • Emmanuel-Xs•100
    @Emmanuel-Xs

    Can you work on the mobile-responsiveness of the site, make sure the images of the social icons are showing on the live site and try as much as possible to match the design. Nice work keep it up.

  • Emmanuel-Xs•100
    @Emmanuel-Xs

    Yes, I would recommend watching webdev simplified from youtube and you can also check out his courses they are amazing.

    I would also like to point out something in your code. you forget about the background-color of the summary-items and try as much as possible to make all your styles in an external stylesheet if possible.

    Marked as helpful
  • Emmanuel-Xs•100
    @Emmanuel-Xs

    Try This

    .button {
    background-color: var(--clr-neutral-270);
}

.button:hover,
.button:focus-visible {
    background: var(--gradient-primary);
}

    Replace my custom properties with yours

    NOTE It is not possible to do transitions on linear-gradient.

    Marked as helpful
  • Emmanuel-Xs•100
    @Emmanuel-Xs

    Okay, let me try your solution out. I created the button eventListener inside the inputs to ensure the user clicks a rating before submitting without writing extra code. However, from what you said, it creates multiple click events for the button that may slow down the browser's overall speed. So, I will do as you say.

    Thank you very much I tried it and it worked. I really appreciated