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

avert some confusion in the tour #1483

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Conversation

cellio
Copy link
Member

@cellio cellio commented Dec 13, 2024

Addresses https://meta.codidact.com/posts/292383.

For most of the tour, you get a "next" button at each step that advances to the next stage. Voting is different, though: when you get to the first answer, the tour tells you to vote on it and gives you a "back" button but no "next" button. This change adds a line that the tour will continue after you vote.

Yes, we're railroading you into voting here, while you can click through the earlier steps without doing anything. We also railroad you when it comes to flagging the second answer, so that feels kind of ok. I tried adding a "next" button that would skip the voting and move on, but I couldn't get it to work with two paths out from this step to the next.

Should we remove the "back" button here, so the only clicky thing is the vote buttons? "Back" works, but I don't know how useful it is, and the rest of the tour doesn't have back-navigation.

@cellio cellio requested a review from a team December 13, 2024 01:44
@cellio cellio added the area: html/css/js Changes to front-end code label Dec 13, 2024
@ArtOfCode- ArtOfCode- merged commit 20dc4d1 into develop Dec 19, 2024
7 checks passed
@ArtOfCode- ArtOfCode- deleted the cellio/tour-tweak-voting branch December 19, 2024 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: html/css/js Changes to front-end code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants