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

better backbutton #57

Merged
merged 2 commits into from
May 12, 2020
Merged

better backbutton #57

merged 2 commits into from
May 12, 2020

Conversation

zeratax
Copy link
Collaborator

@zeratax zeratax commented May 12, 2020

!closes #13
Everything gets saved into a cookie that looks like this:
cookie
and expires after 1 hour.
switching between languages or reloading now also won't reset your currently entered questions.
After answering the last question or waiting one hour everything should be reset again.

it should also now be fairly easy to instead of decrementing or incrementing to do #6

I don't have tested this much

zeratax added 2 commits May 12, 2020 18:59
this allows us to easily switch between languages and even go back
multiple questions
@zeratax zeratax added the enhancement New feature or request label May 12, 2020
@zeratax zeratax requested a review from sparkz9502 May 12, 2020 17:07
@sparkz9502 sparkz9502 merged commit 53a0e80 into master May 12, 2020
@sparkz9502
Copy link
Member

So I tested it locally and it seemed to work fine, but once merged into master it seems to not work properly on the web. For now I've reverted the merge.

@zeratax
Copy link
Collaborator Author

zeratax commented May 13, 2020

hmm not worked properly as in?

@zeratax
Copy link
Collaborator Author

zeratax commented May 13, 2020

tried it on a fork and seems to work fine, did take like a few minutes to update tho
https://zeratax.github.io/leftvalues.github.io/quiz.html

@sparkz9502
Copy link
Member

sparkz9502 commented May 14, 2020

@zeratax In my experience (using Google Chrome), the results error out, with all axes displaying as 50% no matter how the questions are answered. Additionally, it seems it is not possible to go back to the first question after getting to the second question.

@zeratax
Copy link
Collaborator Author

zeratax commented May 14, 2020

oh yeah same for me with firefox, I'll look at that tomorrow. was that the same when you merged it to master?

@sparkz9502
Copy link
Member

Yeah, the same issues occurred.

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

Successfully merging this pull request may close these issues.

Can only go back one question
2 participants