Avalonia icon indicating copy to clipboard operation
Avalonia copied to clipboard

Update all TemplateParts to Match Convention

Open robloo opened this issue 2 years ago • 4 comments

Describe the bug

Certain controls do not properly name their template parts. These should be renamed to follow convention before the 11.0 release as it is a breaking change for the styles.

This is the breaking-change following PR #7911 and #7994.

Also discussed in #7432

I will also close this one, just adding to keep track of it.

Expected behavior

All template parts should start with the "PART_" prefix as is done in WPF and most controls.

Screenshots

N/A

Desktop (please complete the following information):

  • OS: [e.g. Windows, Mac, Linux (State distribution)]
  • Version [e.g. 0.10.0-rc1 or 0.9.12]

Additional context Add any other context about the problem here.

robloo avatar Apr 30 '22 17:04 robloo

@Takoooooo @maxkatz6

I'm waiting on the big styling updates branch to be merged before I do this. However, it still needs to be done before 11.0. If you want, just let me know a good time to open a PR and I can do this quickly.

robloo avatar Jun 24 '22 12:06 robloo

Yes, it should be done after control theme changes, otherwise huge conflicts are unavoidable. Also, not really necessary to have this done in first preview versions.

maxkatz6 avatar Jun 24 '22 22:06 maxkatz6

Now that all the control theme updates are made, I'm going to go through and update template parts within the next week or so.

robloo avatar Aug 10 '22 02:08 robloo

Adding link to a specific discussion: https://github.com/AvaloniaUI/Avalonia/issues/3538#issuecomment-1141275108

robloo avatar Aug 10 '22 02:08 robloo

Well, I had no visibility on the Preview 1 release date or I would have got this in. However, there is a draft PR now.

robloo avatar Aug 21 '22 02:08 robloo