search

ckskr / capstone-project

capstone-project-pied.vercel.app
MIT License
0 stars 0 forks source link

User story 4 // Add radio buttons to form #14

Closed ckskr closed 2 years ago

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
capstone-project ✅ Ready (Inspect) Visit Preview Aug 30, 2022 at 2:20PM (UTC)
UniversalTraveller commented 2 years ago

Hi Claudia! This looks very good in general. 🚀 I just have one little comment about the keys you use to store the value from the radio button.

UniversalTraveller commented 2 years ago

One more thing: I'm not a big fan of calling styled components StyledXY. Take this for example:

` 

                <StyledRadioLabel> Proud</StyledRadioLabel>
            </StyledDiv>`

Isn't it much better readable when it looks like this?

` 

                <MoodButtonLabel> Proud</MoodButtonLabel>
            </MoodButtonBox>`

I made a few suggestions in the comments, but there are some more styled components that could use a more specific name.