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

[iOS] TextDecoration Strikethrough not working on iOS together with LineHeight #11928

Merged
merged 6 commits into from
Nov 18, 2021

Conversation

jsuarezruiz
Copy link
Contributor

Description of Change

Fix issue on iOS Label using TextDecoration Strikethrough together with LineHeight.

Issues Resolved

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

fix11829

Testing Procedure

Launch Core Gallery and navigate to the issue 11928. If text is underline, the test has passed.

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@samhouts samhouts added this to In Review in vCurrent (4.8.0) Aug 26, 2020
@samhouts samhouts added retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. and removed retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. labels Aug 26, 2020
@samhouts samhouts changed the base branch from 4.8.0 to 5.0.0 August 26, 2020 21:27
@samhouts samhouts removed this from In Review in vCurrent (4.8.0) Aug 26, 2020
@samhouts samhouts added this to In Review in vNext+1 (5.0.0) Aug 26, 2020
Copy link
Contributor

@hartez hartez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drop the manual UI test and add an automated platform test instead.


namespace Xamarin.Forms.Controls.Issues
{
#if UITEST
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be automated with a platform test on iOS - we can verify that the Control's AttributedText has the strikethrough attribute after the Label has LineHeight and TextDecoration set.

@samhouts samhouts added retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. and removed retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. labels Nov 2, 2020
@PureWeen PureWeen added the DO-NOT-MERGE-!!! 🛑 This is in progress and needs to be updated before it can be merged. label Nov 5, 2020
@PureWeen PureWeen added this to In progress in v5.0.1 via automation Nov 5, 2020
@PureWeen PureWeen removed this from In Review in vNext+1 (5.0.0) Nov 5, 2020
@samhouts samhouts added this to In Progress in vNext+1 (5.0.0) Nov 5, 2020
@samhouts samhouts removed this from In progress in v5.0.1 Nov 5, 2020
@PureWeen PureWeen removed this from In Progress in vNext+1 (5.0.0) Nov 5, 2020
@PureWeen PureWeen added this to In progress in v5.0.1 via automation Nov 5, 2020
@jsuarezruiz jsuarezruiz removed the DO-NOT-MERGE-!!! 🛑 This is in progress and needs to be updated before it can be merged. label Dec 17, 2020
@jfversluis jfversluis merged commit 4d4c8da into 5.0.0 Nov 18, 2021
v5.0.1 automation moved this from In progress to Done Nov 18, 2021
@jfversluis jfversluis deleted the fix-11829 branch November 18, 2021 21:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Development

Successfully merging this pull request may close these issues.

[Bug] TextDecoration Strikethrough not working on iOS together with LineHeight
5 participants