bikehopper / bikehopper-ui

Friendly bike+transit directions (frontend)
Other
27 stars 1 forks source link

Fix more collapsible leg issues #328

Closed graue closed 6 months ago

graue commented 6 months ago

PR to PR #324 implementing most of my feedback to that PR.

Not implemented because didn't seem blocking: Suggestion to use CSS to set the size (number of lines) of clamped alerts, using overflow: hidden and text-overflow: ellipsis, instead of a character limit

graue commented 6 months ago

for posterity: we did some testing and found that this branch removes much of the slowdown for some reason (possibly related to not rendering twice because of the render call in setState being removed). We think it's shippable now, although there is a bit of stuttering on @abhumbla's phone.