codeforamerica / trailsy

A trip planner and mapper for hiking/biking/riding trail systems.
http://trailsy.herokuapp.com/
30 stars 33 forks source link

site is crashing on ios 6 on safari and stuck on chrome (trailsy.herokuapp.com) #210

Closed katielewis closed 10 years ago

katielewis commented 11 years ago

app begins to load on safari and then safari crashes

on chrome, the screen loads like this: I can still navigate through the list, but I can't do anything else because the header is missing. 09b5f5b0-4679-11e3-9d8a-0a3c65732538

danavery commented 11 years ago

try trailsy-dev. there are some optimizations there from last night.

On Sat, Nov 9, 2013 at 1:00 PM, katie notifications@github.com wrote:

app begins to load on safari and then safari crashes

on chrome, the screen loads like this: I can still navigate through the list, but I can't do anything else because the header is missing. [image: 09b5f5b0-4679-11e3-9d8a-0a3c65732538]https://f.cloud.github.com/assets/3212652/1507440/6925ed2e-4981-11e3-95a4-b8c9b9ebb76a.png

— Reply to this email directly or view it on GitHubhttps://github.com/codeforamerica/trailsy/issues/210 .

katielewis commented 11 years ago

Yep, things are working on both in trailsy-dev, though the map loads really slowly

danavery commented 11 years ago

I've made a slight tweak to simplify the trail lines a little on mobile, which might help with the 4S on iOS 6. I have a 4S with iOS 7 here I'm testing with.

On Sat, Nov 9, 2013 at 1:28 PM, katie notifications@github.com wrote:

Yep, things are working on both in trailsy-dev, though the map loads really slowly

— Reply to this email directly or view it on GitHubhttps://github.com/codeforamerica/trailsy/issues/210#issuecomment-28137418 .

danavery commented 11 years ago

Said tweak is on trailsy-dev as of three minutes ago.

On Sat, Nov 9, 2013 at 1:37 PM, Dan Avery danavery@codeforamerica.orgwrote:

I've made a slight tweak to simplify the trail lines a little on mobile, which might help with the 4S on iOS 6. I have a 4S with iOS 7 here I'm testing with.

On Sat, Nov 9, 2013 at 1:28 PM, katie notifications@github.com wrote:

Yep, things are working on both in trailsy-dev, though the map loads really slowly

— Reply to this email directly or view it on GitHubhttps://github.com/codeforamerica/trailsy/issues/210#issuecomment-28137418 .

katielewis commented 11 years ago

Yep, I think that it is a little faster nw.

danavery commented 11 years ago

There are some more changes on master that may help performance. May. #216. Next time you sync with master, give it a shot.

danavery commented 11 years ago

I've pinned down the source of the long delay on startup on the 4S, I think. fixDuplicateTrailheadTrails(). It will take some time to fix, but should speed startup everywhere if I can speed that part up.

danavery commented 11 years ago

OK, so that wasn't it, but I've streamlined makeTrailDivs quite a bit and re-placed some event handlers (#216) and things are getting much, much better. Not in production yet, but will be soon. More time profiling today.

danavery commented 11 years ago

just pushed more optimizations. more to come.