Vinay Gaba
03/03/2021, 5:35 AMTextField
as the title
slot of a TopAppBar
. I’m actually using a TextField
or a Text
depending on some state (search being active vs not). The thread has more details. Has anyone noticed something similar?Vinay Gaba
03/03/2021, 5:36 AMVinay Gaba
03/03/2021, 5:37 AMVinay Gaba
03/03/2021, 5:37 AMeygraber
03/04/2021, 1:52 AMShowkaseSearchField
actually getting called on line 95? If none of those cases were true, nothing would get shown because there is no else
clauseVinay Gaba
03/04/2021, 2:23 AMTash
03/04/2021, 9:54 AMComponents
. Replaced TopAppBar
title
with ShowkaseSearchField
directly and that shows the text field just fine. I wonder if its some state/composition issue thats making it reach a weird state.Tash
03/04/2021, 9:55 AMTopAppBar
that takes in content
that might work as an alternativeVinay Gaba
03/04/2021, 8:27 PMTash
03/04/2021, 8:30 PMVinay Gaba
03/04/2021, 10:26 PMVinay Gaba
03/05/2021, 7:01 AMSurface
inside the TopAppBar
is what’s causing the issue. I’m going to create my own implementation of the TopAppBar
to unblock myself for now.Tash
03/05/2021, 10:22 AMSurface
could cause issues. My initial hunch was that it had something to do with the TopAppBar
title being inset. Tried with TopAppBar
that takes in content, and used a plain mutableStateOf
to flip between text field / title. https://github.com/airbnb/Showkase/pull/133 odd, but this seems to work 🤔Vinay Gaba
03/05/2021, 4:36 PMcontent
variant of TopAppBar and was able to repro the same issue with it when I added an icon on the right of the title Text. Thanks for investigating this further though!Tash
03/05/2021, 5:43 PMTopAppBar
with the text field when isSearchActive == true
. Tried that out too and works about the same (might have to make some tweaks)Vinay Gaba
03/05/2021, 6:00 PMTash
03/06/2021, 6:40 AMTopAppBar
if the issue is reproducible in a minimal reproVinay Gaba
03/06/2021, 6:45 AM