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

[Bug] ContentPage.ControlTemplate gets not rendered in Wpf (#7671) #7728

Merged
merged 4 commits into from
Nov 16, 2021

Conversation

neolithos
Copy link
Contributor

@neolithos neolithos commented Sep 28, 2019

Description of Change

PageRenderer renderes the first child instead of the Content.

Sorry, I destroyed the branch #7672.

Issues Resolved

API Changes

None

Platforms Affected

  • WPF

Behavioral/Visual Changes

ControlTemplate gets rendered in wpf.

Before/After Screenshots

Before:
PageRendererBefore
After:
PageRendererAfter

PR Checklist

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

@samhouts samhouts added this to In Review in v4.4.0 Sep 28, 2019
@neolithos
Copy link
Contributor Author

Is there a better solution?

Because in the FormsContentLoader it looks like, that someone had take care of ControlTemplates and Pages some how? But I had to change this a line for the layout.

Please, have a closer look to my changes.

{
public class FormsContentPage : FormsPage
{
public FormsContentPage()
{
this.DefaultStyleKey = typeof(FormsContentPage);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Why not using the FrameworkPropertyMetadata?

@samhouts samhouts added in-progress This issue has an associated pull request that may resolve it! and removed in-progress This issue has an associated pull request that may resolve it! labels Oct 2, 2019
@samhouts samhouts removed this from In Review in v4.4.0 Nov 2, 2019
@neolithos
Copy link
Contributor Author

What is missing? What is wrong?

@jsuarezruiz
Copy link
Contributor

@neolithos Could you rebase master to fix the conflict, please?

@neolithos
Copy link
Contributor Author

neolithos commented May 25, 2020

deleted

I found out, what I did wrong. It should be correct, now.

@samhouts samhouts self-requested a review May 28, 2020 17:21
@samhouts samhouts self-assigned this May 28, 2020
@samhouts samhouts removed the request for review from StephaneDelcroix June 2, 2020 23:41
@samhouts samhouts changed the base branch from master to 4.8.0 June 26, 2020 00:14
@samhouts samhouts changed the base branch from 4.8.0 to main July 30, 2020 00:10
@samhouts samhouts added this to In Review in .NET MAUI Backlog Jul 30, 2020
@neolithos
Copy link
Contributor Author

I rebased this a more than ones since 2019.

But how do I solve this:

! [remote rejected]     issue-7671 -> issue-7671 (refusing to allow an OAuth App to create or update workflow `.github/workflows/dotnet-format-daily.yml` without `workflow` scope)

@rmarinho rmarinho removed this from In Review in vNext+1 (5.0.0) Oct 30, 2020
@rmarinho rmarinho modified the milestones: 5.0.0, .NET MAUI Oct 30, 2020
@rmarinho rmarinho removed the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label Oct 30, 2020
@PureWeen PureWeen modified the milestones: .NET MAUI, 5.0.1 Nov 5, 2020
@PureWeen PureWeen added this to In progress in v5.0.1 via automation Nov 5, 2020
Style issues

Layout was applied on the content and not the visual first child

Fix: typo

Shared project fix Xml-Tag

Fix for issue xamarin#7671

Style issues

Layout was applied on the content and not the visual first child

Fix: typo
@jfversluis
Copy link
Member

jfversluis commented Nov 16, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jfversluis jfversluis removed this from In progress in v5.0.1 Nov 16, 2021
@jfversluis jfversluis added this to Issues in Progress in 5.0.0 SR8 (Planning) - Target Date Dec. 15th via automation Nov 16, 2021
@jfversluis jfversluis merged commit a28c1f7 into xamarin:5.0.0 Nov 16, 2021
5.0.0 SR8 (Planning) - Target Date Dec. 15th automation moved this from Issues in Progress to Done Nov 16, 2021
@jfversluis
Copy link
Member

Epic! It took a little while (🤏 ) but finally got to look at this. It works for the Xamarin Community Toolkit sample app! Thanks for this great contribution!

@neolithos
Copy link
Contributor Author

That was long...

@jfversluis
Copy link
Member

At least it wasn't never! 🎉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

[Bug] ContentPage.ControlTemplate gets not rendered in Wpf
8 participants