Skip to content

Reduce search_index.js size #64

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

Closed
Xuhao opened this issue Jan 22, 2014 · 11 comments
Closed

Reduce search_index.js size #64

Xuhao opened this issue Jan 22, 2014 · 11 comments

Comments

@Xuhao
Copy link
Contributor

Xuhao commented Jan 22, 2014

Is there any way to reduce search_index.js file?

@manuelmeurer
Copy link
Contributor

👍 Becomes > 20MB sometimes...

@zzak
Copy link
Member

zzak commented Feb 15, 2014

Reduce in what way? minify, etc?

@manuelmeurer
Copy link
Contributor

AFAICS it's already minified and I guess it has to be that large to support searching for all possible keys. Could it maybe be loaded asynchronously so the user doesn't have to wait for it?

@kgrz
Copy link

kgrz commented Apr 13, 2014

Looked at the code for a bit and understood that any change to how the index file is generated (or an async JS snippet for loading the index file) should actually be incorporated into rdoc. Right @zzak?

@kgrz
Copy link

kgrz commented Apr 13, 2014

@Xuhao FWIW, Gzipping the file will reduce the size by a HUGE margin. This, however, won't work if the files are being served from the filesystem — in which case, having a large file is not really a problem anyway.

@kgrz
Copy link

kgrz commented Apr 13, 2014

Went ahead and implemented that Gzipping functionality into sdoc: kgrz@43c1aad

(Work in progress. )


@zzak should we have something like that?

@manuelmeurer
Copy link
Contributor

👍

@zzak
Copy link
Member

zzak commented Aug 11, 2014

@kgrz Any update on this?

@kgrz
Copy link

kgrz commented Aug 11, 2014

Nope, will give this a another shot this week. The current code doesn't work as expected. Minor file-naming issues. My initial idea was that a gzipped index file should be generated which can be served by the server when the browser supports it. I still have add that part of the code.

@zzak
Copy link
Member

zzak commented Mar 27, 2015

Hey! Sorry this is old news now, but I wanted to point out:

@manuelmeurer
Copy link
Contributor

👍 Thanks for implementing!

@zzak zzak closed this as completed in 37a5395 Apr 23, 2017
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

No branches or pull requests

4 participants