Skip to content

DEV-16535: Add is-land to quire figures, quire-figure.webc regression fixes#893

Closed
anderspollack wants to merge 25 commits intomainfrom
feature/DEV-16535
Closed

DEV-16535: Add is-land to quire figures, quire-figure.webc regression fixes#893
anderspollack wants to merge 25 commits intomainfrom
feature/DEV-16535

Conversation

@anderspollack
Copy link
Contributor

@anderspollack anderspollack commented Dec 19, 2023

Changed

  • Give addWebcShortcode a default value for attributeNames argument
  • Upgrade is-land to 3.0.2
  • Add support for formats: false
  • Wrap quire figure markup in <is-land>

Fixed

  • Figures styles broken with figures wrapped in elements DEV-16772
  • Empty

    elements are being adding before and after

    DEV-16773
  • Video and audio fallback links are displaying online DEV-16774
  • CLI errors with image sequences are breaking preview/build DEV-16775
  • Vite warning thrown in CLI on preview with new is-land work DEV-16777

@anderspollack
Copy link
Contributor Author

@geealbers these fixes should all be good to go -- I had to do some corresponding style work on thegetty/quire-starter-default#42 to account for the new DOM structure from the is-land elements.

@anderspollack anderspollack changed the title DEV-16535: Implement Lazy loading/unloading for canvasPanel and imageService DEV-16535: ADd is-land to quire figures, quire-figure.webc regression fixes Jan 30, 2024
@anderspollack anderspollack marked this pull request as ready for review January 30, 2024 20:36
@anderspollack anderspollack changed the title DEV-16535: ADd is-land to quire figures, quire-figure.webc regression fixes DEV-16535: Add is-land to quire figures, quire-figure.webc regression fixes Jan 30, 2024
@anderspollack anderspollack requested a review from mphstudios March 4, 2024 19:36
cbutcosk added a commit to cbutcosk/quire that referenced this pull request Mar 7, 2024
@mphstudios mphstudios closed this Mar 4, 2025
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.

2 participants