search

ryanfb / cts-cite-driver

Driver JavaScript app for CTS/CITE integration
http://ryanfb.github.com/cts-cite-driver/
1 stars 3 forks source link

AJAX Spinner goes forever on Safari #4

Open ryanfb opened 7 years ago

ryanfb commented 7 years ago

See #2

ryanfb commented 7 years ago

Looking at the console in Safari, this seems to be caused by running up against localStorage limits:

QuotaExceededError: Dom exception 22: An attempt was made to add something to storage that exceeded the quota

So we need to catch this error whenever we write to localStorage. Also, since we have to do the Fusion Tables query for the CTS text anyway, I don't think we really gain anything from memoizing the CTS result to localStorage - at least for Photios, we don't actually take advantage of this memoization.