Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Bug] Icon not correctly centered with text when using FontImageSource #6632

Open
TechWatching opened this issue Jun 23, 2019 · 2 comments
Open
Labels
e/5 🕔 5 help wanted We welcome community contributions to any issue, but these might be a good place to start! p/Android t/bug 🐛 up-for-grabs We welcome community contributions to any issue, but these might be a good place to start!

Comments

@TechWatching
Copy link

Description

In a button, icon is not correctly centered with the text.

Steps to Reproduce

  1. Add a button to a page with a text and an icon using FontImageSource
  2. Set ContentLayout property of the button to "Left,0"
  3. Launch the app an go to the page

Expected Behavior

Icon and text are centered in the button, the icon appears just on the left of the text without space between them.

Actual Behavior

Text is centered in the button but the icon is not, there is a lot of space between text and button.
It seems that ContentLayout property is not taken into account.

Basic Information

  • Version with issue: Xamarin Forms 4.1
  • IDE: VS 2019
  • Platform Target Frameworks: Only tested on Android
    • Android: 9.0
  • Nuget Packages: Nuget from the default Shell templates
  • Affected Devices: All devices

Screenshots

FontImageSource_Issue

The button "SomeText" on the screenshot shows the icon not correctly centered.

Reproduction Link

Github repo : https://github.com/TechWatching/FontImageSourceIssue

@TechWatching TechWatching added s/unverified New report that has yet to be verified t/bug 🐛 labels Jun 23, 2019
@samhouts samhouts added p/Android a/shell 🐚 e/5 🕔 5 and removed a/shell 🐚 s/unverified New report that has yet to be verified labels Jun 25, 2019
@TechWatching
Copy link
Author

Any update on this issue ? It's preventing me from using buttons on my client's project, and I am not fond of our workaround (frame with label + image in it).
I opened the Button code in Xamarin Forms solution to see if I could correct it myself, but I don't know where to look. Moreover I guess that if it's tagged e/5 it may be too much for a newcomer like me on this repo.

@samhouts samhouts added inactive Issue is older than 6 months and needs to be retested help wanted We welcome community contributions to any issue, but these might be a good place to start! up-for-grabs We welcome community contributions to any issue, but these might be a good place to start! labels Jan 10, 2020
@samhouts samhouts added this to the 5.0.0 milestone Aug 13, 2020
@DennisWelu
Copy link

Also ran into this

@samhouts samhouts removed the inactive Issue is older than 6 months and needs to be retested label Sep 18, 2020
@samhouts samhouts removed this from the 5.0.0 milestone Nov 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
e/5 🕔 5 help wanted We welcome community contributions to any issue, but these might be a good place to start! p/Android t/bug 🐛 up-for-grabs We welcome community contributions to any issue, but these might be a good place to start!
Projects
None yet
Development

No branches or pull requests

3 participants