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

feat: ✨ add page template #3

Draft
wants to merge 21 commits into
base: develop
Choose a base branch
from
Draft

Conversation

limwa
Copy link
Contributor

@limwa limwa commented Feb 7, 2021

This pull request's main objective is to create a template that can be used by our tool, along with Jinja (most likely), to create a page for a given quiz.

This is still a work in progress.

This commit introduces an example of what this tool will be able to generate.
It must be noted, though, that this is obviously a prototype and will be changed in the future.
@limwa limwa added help wanted Extra attention is needed WIP Something is still in the making labels Feb 7, 2021
@limwa limwa self-assigned this Feb 7, 2021
@limwa
Copy link
Contributor Author

limwa commented Feb 7, 2021

Click here to see a preview of the example page!

@gwealm
Copy link
Contributor

gwealm commented Feb 7, 2021

I've noticed that the button to clear the answers only supports a full clear. What if we create an extra option on each question to clear the current answer? What do you think?

@limwa
Copy link
Contributor Author

limwa commented Feb 7, 2021

Good point! I'm thinking we put a danger button after the options for the answer. However, if we do that, I think we will need to handle the clear in JavaScript.

@gwealm gwealm force-pushed the develop branch 3 times, most recently from eaa65e0 to 578a677 Compare February 10, 2021 16:27
@limwa limwa added help wanted Extra attention is needed WIP Something is still in the making labels Feb 11, 2021
@limwa
Copy link
Contributor Author

limwa commented Feb 11, 2021

The only thing left now is navigation!

In this commit, the side of the navigation widget was reduced by 1rem, making it a little bit more spaced.
The name of the navigation widget was also changed to the name of the test.
Finally, keyboard navigation in the navigation widget was improved.
@limwa
Copy link
Contributor Author

limwa commented Feb 12, 2021

Navigation is done!

@limwa limwa force-pushed the feature/create-page-template branch from fc711be to ec63c9d Compare February 12, 2021 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed WIP Something is still in the making
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants