tststs / atom-ternjs

JavaScript code intelligence for atom with tern.js and autocomplete-plus
MIT License
627 stars 76 forks source link

Storing the views in a marker's properties is making them unserializable #289

Closed abe33 closed 7 years ago

abe33 commented 7 years ago

Doing a lot of packages development in Atom, I'm using the reload feature a lot, but lately I started getting IDBStore errors when trying to reload Atom, forcing me to force-quit it. By tracking down that error I found that it was atom-ternjs that was causing it.

There's two occurences where you're storing a reference to a view as a property of a marker, but markers are serialized along with their editor, and as is these views aren't serializable as JSON objects, leading to issues when Atom tries to save that in the local storage.

The two occurences are: