Skip to content

Split sandboxed reviewer iframe#5

Open
andrewsanchez wants to merge 1 commit into
pr4289-split/shared-ts-plumbingfrom
pr4289-split/sandboxed-inner-frame
Open

Split sandboxed reviewer iframe#5
andrewsanchez wants to merge 1 commit into
pr4289-split/shared-ts-plumbingfrom
pr4289-split/sandboxed-inner-frame

Conversation

@andrewsanchez
Copy link
Copy Markdown

Source split from ankitects#4289.

Breakdown sections covered:

    1. Sandboxed inner reviewer iframe
    1. Parent/iframe message passing
    1. Typed answer replacement
    1. MathJax support and font CORS handling
    1. Image occlusion support
    1. localStorage persistence shim
    1. Backwards-compatibility shims

Source fidelity:

Follow-up notes from the breakdown:

  • postMessage currently copies card contents; this should be revisited for large cards.
  • The localStorage shim needs more compatibility and size testing.
  • Typed answer parsing may differ slightly for some nc: / cloze cases.

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