-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TextInputLayout] Exposed the style of the ExposedDropDownMenu #1566
base: master
Are you sure you want to change the base?
[TextInputLayout] Exposed the style of the ExposedDropDownMenu #1566
Conversation
Thanks for the PR! I think it would be confusing to have a Since we only want to affect the dropdown's popup background, I think it would make more sense to have an attribute like Would you be interested in trying that? |
Thanks for your feedback. Are you interested to set in the |
We already set |
Moving all the logic that sets the background to the |
I've just updated the PR moving the all the logic that sets the background to the |
Will these be able to customize the ExposedDropDownMenu dropdown's popup background? If so, when this will be updated? |
Currently the ExposedDropDownMenu provided by
DropdownMenuEndIconDelegate
doesn't use theshapeAppearance
defined in the app theme and the corner radius are defined by code.With this change:
shapeAppearance
defined in the app themeexposedDropDownMenuStyle
defined in theTextInputLayout
style with allows to customize theshapeAppearance
/shapeAppearanceOverlay
(and some other properties) of the ExposedDropDownMenu.This change can be tested with the current catalog (changing the
shapeAppearance
).