Skip to content

Conversation

@gewarren
Copy link
Contributor

@gewarren gewarren commented Sep 25, 2020

Fixes #20789. For APIScan compliance.

Preview link.

The position, relative to <xref:System.Windows.Controls.GridViewHeaderRowPresenter>, where the dragging starts.

`relativePos` <xref:System.Windows.Point>\
The position, relative to `header`, where the dragging starts.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wasn't completely sure on this parameter's description.

Copy link

Choose a reason for hiding this comment

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

Good enough for private method.

The position, relative to `header`, where the dragging starts.

`cancelInvoke` <xref:System.Boolean>\
`true` to cancel the reordering; otherwise, `false`.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wasn't completely sure on this parameter's description.

Copy link

Choose a reason for hiding this comment

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

Good enough.

The position, relative to <xref:System.Windows.Controls.GridViewHeaderRowPresenter>, where the dragging starts.

`relativePos` <xref:System.Windows.Point>\
The position, relative to `header`, where the dragging starts.
Copy link

Choose a reason for hiding this comment

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

Good enough for private method.

The position, relative to `header`, where the dragging starts.

`cancelInvoke` <xref:System.Boolean>\
`true` to cancel the reordering; otherwise, `false`.
Copy link

Choose a reason for hiding this comment

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

Good enough.

@gewarren gewarren requested a review from BillWagner October 8, 2020 22:34
@gewarren
Copy link
Contributor Author

gewarren commented Oct 8, 2020

@dotnet/docs Could one of you sign off on this so I can merge it? It's high priority.

@gewarren gewarren merged commit 7097fbb into dotnet:master Oct 8, 2020
@gewarren gewarren deleted the private-wpf branch October 8, 2020 23:06
@gewarren
Copy link
Contributor Author

gewarren commented Oct 8, 2020

Thanks Sam and Tom!

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.

APIScan: Document three APIs in GridviewHeaderRowPresenter

5 participants