Skip to content

Add Label Edit to TreeView #10149

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

Merged

Conversation

SimonZhao888
Copy link
Member

@SimonZhao888 SimonZhao888 commented Oct 19, 2023

Fixes #9956

Proposed changes

  • Adding a shared control of LabelEdit for ListView and TreeView .
  • Add LabelEdit to the TreeView.
  • Update LabelEdit of ListView.
  • Add test cases for LabelEdit of TreeView.

Customer Impact

  • Label edit in TreeView has correct position in accessibility tree.

Regression?

  • No

Risk

  • Minimal

Screenshots

Before

The LabelEdit control of TreeViewNode will be placed as a first child of the TreeView in accessibility tree.
image

After

The LabelEdit control of TreeViewNode will place as a child of the Editing TreeNode in accessibility tree.
image

Test methodology

  • Manual (Insights, Inspect)

Accessibility testing

Test environment(s)

  • 9.0.100-alpha.1.23504.14
Microsoft Reviewers: Open in CodeFlow
Microsoft Reviewers: Open in CodeFlow

@SimonZhao888
Copy link
Member Author

The failure test cases are not related to this PR.

@lonitra
Copy link
Member

lonitra commented Oct 20, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@lonitra lonitra added the waiting-author-feedback The team requires more information from the author label Oct 20, 2023
@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Oct 23, 2023
@lonitra lonitra added the waiting-author-feedback The team requires more information from the author label Oct 23, 2023
@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Oct 24, 2023
@SimonZhao888 SimonZhao888 requested a review from lonitra October 24, 2023 02:39
@lonitra lonitra added the waiting-author-feedback The team requires more information from the author label Oct 24, 2023
@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Oct 30, 2023
Copy link
Member

@lonitra lonitra left a comment

Choose a reason for hiding this comment

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

LGTM

@lonitra lonitra added the ready-to-merge PRs that are ready to merge but worth notifying the internal team. label Oct 30, 2023
@SimonZhao888 SimonZhao888 merged commit e073460 into dotnet:main Oct 31, 2023
@ghost ghost added this to the 9.0 Preview1 milestone Oct 31, 2023
@ghost ghost removed the ready-to-merge PRs that are ready to merge but worth notifying the internal team. label Oct 31, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Nov 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Label edit in TreeView has incorrect position in accessibility tree
4 participants