Skip to content
Snippets Groups Projects

Fix archive images.

Merged Klaas Kliffen requested to merge tickets/184_fix_archive_images into master
All threads resolved!

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Mattia Mancini
  • Mattia Mancini
  • Just few implementation comments and syntax. Please take them lightly I dont do js editing in ages.

    Also I have noticed there little to no use of await. Is there a reason?

    • Resolved by John Swinbank

      I'm a bit confused with what's going on here. esap-general#184 (closed) just changed some URLs, so it's not obvious why it necessitates such a big diff here.

      I like the idea of fixing image sizes, but beyond that it's hard to see exactly what's changed here (if anything) since the diff is full of formatting changes.

  • It's mainly the ESAP logo is now fixed in the top left corner (it would change something depending on the archive) and those would again be fetched from the ALTA archive. And since the images "are out of our control" I wanted to add a safeguard so that the images would be more or less the same size (which was a valid point made by Nico) and additionally I added an alt text in case it goes missing.

  • Klaas Kliffen resolved all threads

    resolved all threads

  • John Swinbank approved this merge request

    approved this merge request

  • John Swinbank resolved all threads

    resolved all threads

  • merged

  • John Swinbank mentioned in commit 9605fcbd

    mentioned in commit 9605fcbd

  • Please register or sign in to reply
    Loading