Skip to content
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

Parsing of the .json in the background #12

Open
MagTun opened this issue Apr 11, 2021 · 1 comment
Open

Parsing of the .json in the background #12

MagTun opened this issue Apr 11, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@MagTun
Copy link
Contributor

MagTun commented Apr 11, 2021

Currently the extension has to be kept open to finish the export. For big courses (ex: with 5000+ words) it takes a while and it's a bit annoying as we can't use our browser in the meantime (as this closes the extension). So it would be great if the whole run() including the .json parsing would happen in the background.

I tried to solve this by moving everything in popup.js (except the main document.addEventListener('DOMContentLoaded',...) ) into background.js but without any success (even when using chrome.runtime.onMessage to start the run()).

So I am sorry: for this one, I can only request the feature, and give you a little help to implement it.

@raineorshine
Copy link
Owner

Thanks for creating an issue! I agree this would make for a better user experience. Maybe I can investigate when I have some free time.

@raineorshine raineorshine added the enhancement New feature or request label Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants