Skip to content
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

[Android] Changes updating ImageButton Padding to avoid size issues #14905

Merged
merged 5 commits into from May 9, 2023

Conversation

jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented May 3, 2023

Description of Change

Changes updating Android ImageButton Padding to avoid sizing issues.

Before
ib-padding-before
Notice the external padding/border issue in addition to the sizing problems.

After
ib-padding-after

To validate the changes use the new sample added to the ImageButton page in the .NET MAUI Gallery.

Issues Fixed

Fixes #7927
Fixes #13101

@jsuarezruiz jsuarezruiz added t/bug Something isn't working platform/android 🤖 control-button Button, ImageButton labels May 3, 2023
@ghost ghost added the area/controls 🎮 Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label May 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

Thank you for your pull request. We are auto-formatting your source code to follow our code guidelines.

@jsuarezruiz jsuarezruiz requested a review from hartez May 9, 2023 13:56
@hartez hartez merged commit 1bbe79d into main May 9, 2023
29 checks passed
@hartez hartez deleted the fix-7927 branch May 9, 2023 21:22
rmarinho pushed a commit that referenced this pull request May 30, 2023
…14905)

* Changes updating Android ImageButton Padding

* Updated tests

* Auto-format source code

---------

Co-authored-by: GitHub Actions Autoformatter <autoformat@example.com>
Vetle444 added a commit to DIPSAS/DIPS.Mobile.UI that referenced this pull request Jul 3, 2023
[Android] dui:ImageButton now fixes padding issue: dotnet/maui#14905
@github-actions github-actions bot locked and limited conversation to collaborators Dec 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/controls 🎮 Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor control-button Button, ImageButton platform/android 🤖 t/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Padding on ImageButtons works wrong on Android ImageButton does not respect the height and width given to it
2 participants