Update HomeScreen.kt #978
Update HomeScreen.kt #978
Conversation
Removed fillMaxWidth() from CenterAlignedTopAppBar's title because when switching between My garden and Plant list filter icon pops up and pushes CenterAlignedTopAppBar to the left.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
| @@ -169,12 +169,12 @@ private fun HomeTopAppBar( | |||
| CenterAlignedTopAppBar( | |||
| title = { | |||
| Row( | |||
| Modifier.fillMaxWidth(), | |||
| Modifier, | |||
There was a problem hiding this comment.
You can delete this line since the default is Modifier. In fact, you can probably just remove the entire row since an Arrangement.Center doesn't do anything if it's only matching the child's size. Can you try deleting the row and just having the text be the title to verify that looks right?
There was a problem hiding this comment.
You can delete this line since the default is Modifier. In fact, you can probably just remove the entire row since an Arrangement.Center doesn't do anything if it's only matching the child's size. Can you try deleting the row and just having the text be the title to verify that looks right?
yes sir you are right it still working same.
Yes even after removing the Row composable its working good that means Row composable is useless we can remove it
|
Sorry, I was off work Thursday and Friday, but I've merged now. Thanks for your contribution! |
Thanku so much sir. You reviewed it that means a lot thanku for giving me my first contribution to open source. |
Removed fillMaxWidth() from CenterAlignedTopAppBar's title because when switching between My garden and Plant list then filter icon pops up and pushes CenterAlignedTopAppBar to the left
------------->This is the app before Change
.
.
.
.
------------->This is the app after Change