search

remulasce / metroapp

Realtime arrival notification utility for LA Metro
2 stars 0 forks source link

IOS MultiArrivalView spacing #187

Closed nighelles closed 9 years ago

nighelles commented 9 years ago

The sizing of the UITableViewCell elements nominally is working, but it oddly uses tiles that are too large or too small if it's not the first thing you searched for. I think its dequeueing a cell for reuse, and not updating how large it should be.

nighelles commented 9 years ago

Still odd, seems almost like an indexing problem, but the indexing seems fine.

Doesn't seem like a race condition either, there's no way we're taking that long to render the array

nighelles commented 9 years ago

Further oddness, the log doesn't seem to agree with what's being displayed, even though they're both being generated from the same place. I.E. Log says 3,2,5,2. but MultiArrivalTripView is displaying like 2, 5, 3 with odd spacing.

nighelles commented 9 years ago

Extra trips aren't updating time either, detritus from previous render?

nighelles commented 9 years ago

Looks like persistance of extra fields when a multiarrivaltrip farther down the list doesn't contain as many elements

nighelles commented 9 years ago

Fixed with a careful look at how exactly cells are reused. Old programmatically generated labels still existed and were populated with garbage.