search

robinhood / ticker

An Android text view with scrolling text change animation
https://medium.com/robinhood-engineering/hello-ticker-20eaf6e51689
Apache License 2.0
4.38k stars 462 forks source link

wrap_content doesn't work when set value from a larger number to a smaller number #15

Closed BaronZ closed 8 years ago

BaronZ commented 8 years ago

wrap_content doesn't work when set value from a larger number to a smaller number. lets assume || is a textview, change number from 1234567 to 123, the width of the textview turns out like below

|1234567| |123　　 |

What I think it supposed to be is like below

|1234567| |123|

peacepassion commented 8 years ago

I also found this problem. I would like to post a gif later.
tick_bug

BaronZ commented 8 years ago

@peacepassion urs looks fine

jinatonic commented 8 years ago

Ah, I see the problem. We should be re-measuring after the animation completes (we don't re-measure for each frame of the animation) but that doesn't seem to be happening for you. I'll investigate this. Thanks for reporting!