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 solutions

  • Submitted


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

    Proud of turning a list of radio inputs into 'buttons' that returned their assigned values to be used in the JS function.

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

    As my first JavaScript project, I'd love any feedback on my script. Is there a more efficient/better way to do what this project requires?

    Also, I made the form 'required' with an if statement within the function

    if (radios !== null) {
            formStart.style.display = "none";
            formFinish.style.display = "block";
            selectedMessage.innerText = `You selected ${radios} out of 5`
        }
    

    As the button on the form isn't a submit button, I believe the normal method of adding 'required' to the HTML inputs wouldn't work. Is my workaround ok or once again is there a better way?

  • Submitted


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

    I'd appreciate feedback on my @media rules, as although everything is working as it should across screen widths, I think the code could be simplified.

    Also, for the staggering of the 5 star review boxes, I just added nth-of-type rules with margin and negative margin on either side. I assume this should instead be done with a position rule or something like that instead. The code looks like:

    nth-of-type(2) {
            margin-right: 40px;
            margin-left: -40px
    }
    

    Similarly for the bottom 3 testimonial boxes.

    Any other feedback is welcome.

  • Submitted


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

    I feel I have got the design to look almost exactly the same as the brief, which is great considering I didn't have access to the design files. Only issue is next time I need to remember to consider responsiveness earlier in the process.