Skip to content

Update viz

Archer, Althea Anne requested to merge update-viz into main

To test, npm install and npm run dev (no need to run the R scripts unless you want to, they're in the R_src directory)

This MR does:

  1. Update the way background images fit so that they're always showing the full image rather than getting cut off
  2. Update placement of png elements on top of background images
  3. Update the text to drop the work "ranger" (wrt SME advice)
  4. Update the button styling (#4 (closed))
  5. Change to grid setting (close #12 (closed))
  6. Includes !18 (merged) changes, such as including mobile layout, grid layout, etc (close #13 (closed))
  7. Includes references and placeholder for authorship (closes #5 (closed) and #6 (closed))

This MR doesn't:

  1. Read images from s3 #11 (closed)
  2. Change button styles to something more fitting with theme #4 (closed)
  3. Fix the fact that the background image div size is not consistent (based on js logic, the size of the background image div is very small on a few key places) #12 (closed)
  4. Fix the fact that mobile is not being recognized? #13 (closed)
  5. Fix the refresh problem when you're in the CYOA part #14 (closed)
  6. Add alt text #15 (closed)
  7. References links aren't working #16 (closed)
  8. Fix the png images in place on the background image (as one png per page)

Things I don't love but are maybe ok for MVP FIXED

  1. The large gap at the top of the page between the short title and the narrative text (top screenshot), I tried fixing this with css grid properties but didn't get any quick changes to fix it. I would prefer if the grid fit better so that buttons didn't overlap the images, and that gap wasn't so big.
  2. I could redo these illustrations (second screenshot) so that the landscape looks better and isn't so small

image

image

Edited by Archer, Althea Anne

Merge request reports

Loading