Skip to content

Conversation

@guimafelipe
Copy link
Member

Fixes Issue

Main PR

Description

This PR is to add an document file about the steps for implement an new automation property, based on the work done for the heading level property, to be used in the future as reference.

Customer Impact

This should help other developers to understand how an automttion property should be implemented.

Regression

Testing

Risk

@guimafelipe guimafelipe requested a review from a team as a code owner October 14, 2021 17:47
@ghost ghost added the PR metadata: Label to tag PRs, to facilitate with triage label Oct 14, 2021
@ghost ghost requested review from SamBent, fabiant3 and ryalanms October 14, 2021 17:47
@guimafelipe guimafelipe changed the title Adding documentation for how to implemente a new Automation Property Adding documentation for how to implement a new Automation Property Oct 15, 2021
protected override System.Windows.Rect GetBoundingRectangleCore() { throw null; }
protected override string GetClassNameCore() { throw null; }
protected override string GetNameCore() { throw null; }
protected override bool IsDialogCore() { throw null; }
Copy link
Member

Choose a reason for hiding this comment

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

Redundant?

Copy link
Member Author

Choose a reason for hiding this comment

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

This was one line that was missing in the reference when I implemented this AutomationProperty. I will remove it as it is not related to the documentation itself.

@guimafelipe guimafelipe merged commit 7e00bae into main Oct 25, 2021
@vishalmsft vishalmsft deleted the add.window.isdialogcore branch February 15, 2022 05:34
@ghost ghost locked as resolved and limited conversation to collaborators Apr 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

PR metadata: Label to tag PRs, to facilitate with triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants