Fix search auto entered

DroidKaigi / conference-app-2020

The Official Conference App for DroidKaigi 2020 Tokyo

https://droidkaigi.jp/2020/en/

Apache License 2.0

774 stars 327 forks source link

Closed sekiya9311 closed 4 years ago

sekiya9311 commented 4 years ago

The cause of this bug is call invalidateOptionsMenu() in toolbar.doOnApplyWindowInsets {} when query text changed.
So, changed place of call invalidateOptionsMenu() from toolbar.doOnApplyWindowInsets {} to toolbar.doOnLayout {}
We can use toolbar.doOnNextLayout {} insted of toolbar.doOnLayout {} and resolved too. But, I don't know which is better. 😥

Before	After

jmatsu-bot commented 4 years ago

Generated by :no_entry_sign: Danger

jmatsu-bot commented 4 years ago

No issue was reported. Cool!

Generated by :no_entry_sign: Danger

takahirom commented 4 years ago

Thank you! You found the cause of this!