search

navcoin / WhisperWallet

Whisper Wallet
5 stars 4 forks source link

Android AccountsTab ScrollView does not scroll #95

Closed aguycalled closed 2 years ago

aguycalled commented 2 years ago

untitled

tsejerome commented 2 years ago

I think it's not supposed to be scrolling as the screens has shown all of them alrdy and this is how native android looks but I will double check tmr

aguycalled commented 2 years ago

iOS looks in a more natural way. the spinner showing on top of the items looks weird imo

tsejerome commented 2 years ago

I find it weird to have a pull to refresh at the lower half of the screen, how about we just have a full page refresh button at the top right corner next to the "..." menu?

aguycalled commented 2 years ago

it does not make sense to me. the bottom part is what gets scrolled. users can have an arbitrary amount of private tokens and will need to scroll through them

tsejerome commented 2 years ago

@aguycalled is there any app that has half page scrollview that you cud send over for us to reference?

aguycalled commented 2 years ago

I can't think now of any app that has half page scroll view, but I think it is what makes more sense in our ui. it looks good to me on ios

tsejerome commented 2 years ago

Alrighty, but I believe the scroll is perfectly normal for android users ( i have been seeing it like this for years )

tsejerome commented 2 years ago

Conclusion: adding a reload button at the top