tulios / json-viewer

It is a Chrome extension for printing JSON and JSONP.
https://chrome.google.com/webstore/detail/json-viewer/gbmdgpbipfallnflgajpaliibnhdgobh
MIT License
3.31k stars 678 forks source link

Performance improvements #225

Open richiksc opened 5 years ago

richiksc commented 5 years ago

These changes improve performance by: a) Enabling tree-shaking by using ES6 imports b) Short-circuiting the JSON check if the correct content-type is served c) Short-circuiting to false if the JSON body does not start with '{' or '[' d) Lazy-loading the rest of the viewer only if the JSON check passes.

This should help with tulios/json-viewer#142.

richiksc commented 5 years ago

@tulios?

mikhoul commented 5 years ago

Does somebody could update the extension on the Chrome webstore with the latest changes/commits including those ones ?

Regards :octocat:

jonluca commented 5 years ago

Pinging @tulios again.

richiksc commented 3 years ago

@tulios Any updates on whether this might be merged?