Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Star Question Type #16

Merged
merged 1 commit into from
Apr 18, 2023
Merged

Add Star Question Type #16

merged 1 commit into from
Apr 18, 2023

Conversation

miles-grant-ibigroup
Copy link
Collaborator

This PR adds a new question type, star. Similar to smiley slider.

@vercel
Copy link

vercel bot commented Apr 6, 2023

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

Name Status Preview Comments Updated (UTC)
atlrides-survey ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2023 2:30pm
qa-survey ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2023 2:30pm
st-survey ✅ Ready (Inspect) Visit Preview 1 resolved Apr 17, 2023 2:30pm

Copy link
Contributor

@amy-corson-ibigroup amy-corson-ibigroup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! A few questions but probably more for me than you!

components/QuestionRenderer.tsx Show resolved Hide resolved
components/stories/Stars.story.tsx Show resolved Hide resolved
components/Stars.tsx Show resolved Hide resolved
Copy link
Contributor

@AdrianaCeric AdrianaCeric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The star question looks awesome, just one a11y comment and a styling suggestion

components/Stars.tsx Show resolved Hide resolved
display: none;
}

.star {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could we also add an outline around the stars

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried a few variations of this. Unfortunately the way we render the stars we can't get an outline or shadow on the star itself, only on the box around the star. This could be resolved with clip-path, or totally re-doing how we render the stars...

I do agree though the styling needs some work. Not sure which direction to go... Borders would match the radio button style, maybe editing the svg is the way to go?

You have any other ideas

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opened #18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants