Skip to content

Feat: card popup window #26

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Feat: card popup window #26

wants to merge 7 commits into from

Conversation

micwallace
Copy link
Contributor

@futantan As discussed this adds an option to open tokenscript actions in a popup window rather than an iframe modal.
This speeds up the loading process significantly, since script info and token data can be cached. It also enables local storage support for cards opened this way.

At the moment the default is false, so the extension will continue to work as-is unless the user explicitly turns this feature on via the extension options page.

Open TS viewer window to resolve proxy.
This allows us to break out of the extension sandbox.
Copy link

vercel bot commented Nov 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
buy-me-a-beer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 22, 2024 3:33am
tlink-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 22, 2024 3:33am

@micwallace
Copy link
Contributor Author

Note: Please merge recaptcha PR first, since this branch shares code with it.

@micwallace micwallace requested a review from futantan November 21, 2024 05:27
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