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

reading xlsx progress bar #632

Open
firas3d opened this issue Apr 16, 2017 · 6 comments
Open

reading xlsx progress bar #632

firas3d opened this issue Apr 16, 2017 · 6 comments
Labels

Comments

@firas3d
Copy link

firas3d commented Apr 16, 2017

First, cannot thank you enough for all the efforts.

I am using js-xlsx to read the excel file with multiple sheets inside Adobe ExtendScript, and have a small request:

Reading the xlsx is taking 10 to 20 seconds, and would be great if i can get a feedback percent,
so I map it to a progress bar inside the UI.

Currently, reading xlsx freezes the application till it finishes.

image

Suggestion: maybe read sheet by sheet or row by row, can solve it.

Hope this feature request will be helpful in other js applications.

@SheetJSDev
Copy link
Contributor

The web browsers have a neat feature called Web Workers that let you run tasks in the background without blocking the UI. That's what http://oss.sheetjs.com/ and http://oss.sheetjs.com/js-xlsx use to ensure the frontend doesn't just stop. There may be a way to do something similar in extendscript.

As for evented read, the major hurdle is the file format. XLSX uses ZIP files and you have to read in the entire file before you can properly unzip. We will think more about this

@firas3d
Copy link
Author

firas3d commented Apr 16, 2017

The freezing till finish reading is acceptable, as long as we have visual feedback (like progress bar)
Maybe jszip.js can send a progress?

@stof
Copy link

stof commented Jul 26, 2017

sending a progress won't help, as the UI would render it only at the end (as it is frozen)

@dingchaoyan1983
Copy link

I want this feature also

@ghost
Copy link

ghost commented Jul 23, 2019

It will help if you are using sheet.js correctly, in a web-worker.

@thrnd
Copy link

thrnd commented Mar 1, 2021

Even if you are using web workers its important to give feedback to user. It would be super useful to be able to show progress (every 100-500 rows would be enough i think).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants