search

ajones05 / seearound.me-ionic

0 stars 0 forks source link

Design updates A: expanded preview #156

Closed ajones05 closed 7 years ago

ajones05 commented 7 years ago

A few issues with expanded preview. The biggest is that tapping on a user name from here causes a critical issue: it appears briefly to load, but does nothing. After that I can still move the screen, but there's no way to close this view or escape without closing the app. A few other issues shown below image

ajones05 commented 7 years ago

Also if first name is longer than [however many characters it would take to run into the options icon, then truncate and add ..."]. This example shouldn't be relevant once it's first name only, but basically so this type of issue doesn't happen: image

abdulhafeez commented 7 years ago

@yuriyua Can you please restrict post.Name to first name only? I think this should be done on server side.

abdulhafeez commented 7 years ago

@ajones05 other issues fixed.

yuriyua commented 7 years ago

@abdulhafeez done https://github.com/ajones05/seearound.me/wiki/Mobile-Api-Reference/_compare/7ad9d8e7c7b36e9ca715f1ad14d83ced1529f47e...a226ea9d3c9fbae5e2a8924b801ac91fb0ed4c8d

http://www.seearound.me/admin/mobile-api/post/token/LLCVS8IMHBDQAPGXEB2SYSV48AX3CBN1Y2VQAFETRTK3I31ZE2SCVPG51D7DPXDN/post_id/4615/submit

http://www.seearound.me/admin/mobile-api/myposts/token/LLCVS8IMHBDQAPGXEB2SYSV48AX3CBN1Y2VQAFETRTK3I31ZE2SCVPG51D7DPXDN/latitude/37.81193351641144/longitude/-122.2842006693827/submit

http://www.seearound.me/admin/mobile-api/request-nearest/token/LLCVS8IMHBDQAPGXEB2SYSV48AX3CBN1Y2VQAFETRTK3I31ZE2SCVPG51D7DPXDN/latitude/37.81193351641144/longitude/-122.2842006693827/submit

abdulhafeez commented 7 years ago

Thanks @yuriyua