Design comparison
Solution retrospective
The code probably It's a mess, so sorry for that. First, I couldn't put the classes where It was supposed to be, so I put kinda close. I didn't know how to reduce the size of <submit>, tried several times but didn't manage to do it. So I let how I managed to get. And second, I managed to change the first container to the second container using JavaScript, but couldn't put the number chosen in <span id="score">. The site reads the number, however the textContent don't get the value, tried several times changing with no sucess. I put the lines 24 and 29 of the JavaScript as a commentary.
Community feedback
- @GowsalayadeviPosted over 2 years ago
Hi, You could use "flex-shrink" to reduce the size of the button and on Line 24 // score.textContent = "value", it should be // score.textContent = value .
Thanks, Gowsalaya
0@Kenzar-SanPosted over 2 years ago@Gowsalayadevi Hi, the score.textContent = value didn't change with or without the quotation, tried several ways to make it work. Tried using flex-shrink but It didn't work, probably something on the css is stopping but dont know what is. Thanks for the help.
0 - @besttlookkPosted over 2 years ago
Hi There are are points i like to point out:
- There is little gap between number and button.
- Number selected does not show in the "thank you" page.
- numbers look too far apart.
Good luck
#happyCoding
0@Kenzar-SanPosted over 2 years ago@besttlookk Yes, I noticed that and have talked about on my feedback, couldn't put the closer like I wanted. There is any tips to change on the code?
0
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord