Fixed
Status Update
Comments
vi...@google.com <vi...@google.com> #2
Hi Ed, Thank you so much for these suggestions. I've been reviewing them and merging them in. Hopefully it should be live. I've included a thank you note too in the article.
vi...@google.com <vi...@google.com> #3
Great! Thanks a lot, I'll look for the live updates soon!
li...@gmail.com <li...@gmail.com> #4
I am very sorry to reply this so late. i thought it was very clear in comment #1 .
This is not a bug, just want to add a new feature.
In my case, i need to expand the height of the toolbar, and we want the menu button to be vertically in center of this toolbar, but buttonGravity api only accept Gravity.TOP or Gravity.BOTTOM, so we can only achieve it in a hack way ( set the value of buttonGravity to Gravity.CENTER_VERTICAL ), and i push my code tohttps://github.com/lixi0912/ToolbarExpandHeightDemo
This is not a bug, just want to add a new feature.
In my case, i need to expand the height of the toolbar, and we want the menu button to be vertically in center of this toolbar, but buttonGravity api only accept Gravity.TOP or Gravity.BOTTOM, so we can only achieve it in a hack way ( set the value of buttonGravity to Gravity.CENTER_VERTICAL ), and i push my code to
vi...@google.com <vi...@google.com> #5
We have passed this defect onto the development team and will update this issue with more information as it becomes available.
Description
Version used: 1.1.0-alpha04
Theme used: Theme.MaterialComponents.DayNight.NoActionBar
Devices/Android versions reproduced on: android pie
- When I set toolbar_maxButtonHeight, since buttonGravity can only be set to top and bottom, it does not meet the CENTER_VERTICAL requirements of the design.