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

New design to assessment page #26

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

Conversation

mfsilva22
Copy link
Contributor

@mfsilva22 mfsilva22 commented Feb 13, 2022

This updates the design of the assessment gallery, and brings the assessment text previously existing from inside the example course repo into the marketing page.

image

Still to be done:

  • Decide if the images used for each assessment should be stored inside of each corresponding assessment folder, or at another common folder
  • Learn how to add the images properly, once the location is defined
  • Fix the links "Read more" and "try it now!" to the bottom of the assessment card, instead of at the end of the text.
  • Learn how to use Latex
  • Decide what to do with the comments for each question for each assessment example. I am inclined to add an image of the text. I don't think the current text/arrangement looks good
  • Create component (?) which stores URL from PrairieLearn for the assessments. As of now, links to assessments are included in different files, and it may be useful to have these links all in one location, so that future updates are easier.
  • Go over the text in each page, and make small improvements, fix links, etc
  • Once this page is done, we should update the actual questions inside the PL repo, to replace the link to GitHub text with the new website (or no more linking)

@vercel
Copy link

vercel bot commented Feb 13, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/prairielearn/marketing/HcoxTcuZBbdTgQBMdPZvkwz2idkN
✅ Preview: Failed

[Deployment for cbd0e5e failed]

@mfsilva22 mfsilva22 marked this pull request as draft February 13, 2022 00:42
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.

1 participant