Skip to content

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

Merged
hartez merged 5 commits intomainfrom
fix-7927
May 9, 2023
Merged

[Android] Changes updating ImageButton Padding to avoid size issues#14905
hartez merged 5 commits intomainfrom
fix-7927

Conversation

@jsuarezruiz
Copy link
Copy Markdown
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 area-controls-button Button, ImageButton labels May 3, 2023
@ghost ghost added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label May 3, 2023
@github-actions
Copy link
Copy Markdown
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 mandel-macaque May 4, 2023 07:34
@jsuarezruiz jsuarezruiz requested a review from hartez May 9, 2023 13:56
@hartez hartez merged commit 1bbe79d into main May 9, 2023
@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
@Eilon Eilon removed the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label May 10, 2024
@samhouts samhouts added the fixed-in-8.0.0-preview.5.8529 Look for this fix in 8.0.0-preview.5.8529! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-controls-button Button, ImageButton fixed-in-8.0.0-preview.5.8529 Look for this fix in 8.0.0-preview.5.8529! 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

4 participants