Ecotrust / bioregion-discovery

Ecotrust's Bioregion Discovery Portal
http://bioregions.apps.ecotrust.org
4 stars 1 forks source link

explanation rather than spinner #30

Closed perrygeo closed 12 years ago

perrygeo commented 12 years ago

during the creation process and the reporting process we might prefer to offer the user an explanation of what's happening 'behind the scenes' rather than just giving them a spinner

Google Code Info: Issue #: 30 Author: sfletche@gmail.com Created On: 2011-07-19T22:08:11.000Z Closed On: 2011-08-04T22:34:01.000Z

perrygeo commented 12 years ago

need to come up with text as well as how best to present the text.

not sure if this requires the async app or not...

Google Code Info: Author: sfletche@gmail.com Created On: 2011-07-19T22:44:33.000Z

perrygeo commented 12 years ago

This is actually pretty tough to do correctly since much of the editing/waiting spinner behavior is hardcoded. The easiest way is to fork kmlEditor.js and override the string (and also need to change the string in the show.html template). This is a quick fix and could lead to future maintenance problems as lingcod evolves (to the extent that we keep the bioregions virtualenv up to date with lingcod trunk).

In summary, its fixed but its a horrible hack and I'm not happy about it.

Google Code Info: Author: perrygeo...@gmail.com Created On: 2011-08-04T22:34:01.000Z