Allow to specify type on MatIconButton. Fixes #315 #325
Conversation
- Material icon button changes - set type in Material autocomplete Thanks!
peterblazejewicz
force-pushed
the
fix/315
branch
from
ad9a45d
to
402fb2a
Compare
|
👉 View analysis in DeepCode’s Dashboard |
| /// "button", "reset", "submit" | ||
| /// </summary> | ||
| [Parameter] | ||
| public string Type { get; set; } |
There was a problem hiding this comment.
Might be good to set the default to "button" ??
public string Type { get; set; } = "button";
src/MatBlazor/Components/MatAutocompleteList/MatAutocompleteList.razor
Outdated
Show resolved
Hide resolved
|
@peterblazejewicz Thanks for contributing. Sorry it has taken a long time to get to your PR. Please review my notes and if you wish to edit your PR then I will merge into the develop branch. Also I recommend you update the DemoIconButton to show examples of use of the new functionality: https://github.com/SamProf/MatBlazor/blob/develop/src/MatBlazor.Demo/Demo/DemoMatIconButton.razor |
No need to commit this file, it will be recompiled at build
|
@enkodellc no worries, times we are living in :) I'll review the current library details, then update this PR (or just close as not relevant anymore). Anyway, I'll post here. Thanks! |
|
@peterblazejewicz , |
Thanks!