search

FirebaseExtended / mlkit-material-android

ML Kit Showcase App with Material Design
Apache License 2.0
491 stars 143 forks source link

Use string templates #5

Closed thatfiredev closed 5 years ago

thatfiredev commented 5 years ago

This PR should:

thatfiredev commented 5 years ago

@zhouyiself I wasn't aware of it, thanks for pointing me to that thread.

I've updated my PR to keep findViewById and use string templates.

thatfiredev commented 5 years ago

Thanks @zhouyiself

On a side note: maybe use "Squash and merge" next time? Just to keep the commit history clean.

zhouyiself commented 5 years ago

Right, clicked too fast...