search

Quartz / Chartbuilder

A front-end charting application that facilitates easy creation of simple beautiful charts
http://quartz.github.io/Chartbuilder
MIT License
2.1k stars 354 forks source link

Loading previous chart gives an error #177

Closed mhkeller closed 9 years ago

mhkeller commented 9 years ago

The button is available but when clicked it throws r.model is null. When I inspect local storage that's also null. I'm in FF 39.0 but @joannaskao said she saw something similar in the Chrome 43.0.2357132

screen shot 2015-07-10 at 1 03 57 pm

nsonnad commented 9 years ago

This is because you have no chart loaded into local storage. Once the timer runs out and you start editing, it will be saved on any change and the button will work.

But the button shouldn't appear if there's nothing in local storage, so I've got a PR for that. https://github.com/Quartz/Chartbuilder/pull/181

nsonnad commented 9 years ago

Fixed at fef27005133a8c36a6386197e78115831f7da08e