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

Revert Inline breaking change #12638

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

MrJul
Copy link
Member

@MrJul MrJul commented Aug 22, 2023

#12624 introduced a binary breaking change undetected by the API diff tool: the public field Inline.TextDecorationsProperty has a different return type.

Fixed by reverting the declaration to StyledProperty, while keeping the new behavior.

@MrJul MrJul force-pushed the fix/inline-breaking-change branch from 19c5da7 to 495e2bb Compare August 22, 2023 22:43
@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 11.0.999-cibuild0038700-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@robloo
Copy link
Contributor

robloo commented Aug 23, 2023

I wonder if this should have an additional comment or be added to #12276. Otherwise, it will probably be forgotten about and the API never switched back to AttachedProperty.

Edit: I need to read... you have the comment. And #12276 is already referenced here in the comment so it should be seen in the future.

@maxkatz6 maxkatz6 added this pull request to the merge queue Aug 23, 2023
Merged via the queue into AvaloniaUI:master with commit a18ad58 Aug 23, 2023
6 checks passed
@MrJul
Copy link
Member Author

MrJul commented Aug 23, 2023

Opened an issue to track the API Compat problem: dotnet/sdk#34864

@MrJul MrJul deleted the fix/inline-breaking-change branch August 23, 2023 12:09
grokys pushed a commit that referenced this pull request Sep 28, 2023
@MrJul MrJul added the backport-candidate-11.0.x Consider this PR for backporting to 11.0 branch label Sep 30, 2023
grokys pushed a commit that referenced this pull request Oct 2, 2023
@grokys grokys added backported-11.0.x and removed backport-candidate-11.0.x Consider this PR for backporting to 11.0 branch labels Oct 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants