Skip to content

Conversation

@Danil-Andrianov
Copy link
Contributor

@Danil-Andrianov Danil-Andrianov commented Jan 28, 2022

Partially implements #3421

Proposed changes

  • Updated "SupportsUiaProviders" flag.
  • Added and implemented toolStripContainer accessible object
  • Added unit tests

Customer Impact

Before update:
InspectWithout
ToolStripContainer1

After update:
InspectWith
ToolStripContainer1

Regression?

  • No

Risk

  • Minimal

Test methodology

  • Unit tests
  • CTI team

Accessibility testing

  • Narrator
  • Inspect
  • Accessibility Insights

Test environment(s)

  • Microsoft Windows [Version 10.0.22000.318]
  • .NET Core SDK: 7.0.0-alpha.1.21562.1
Microsoft Reviewers: Open in CodeFlow

@Danil-Andrianov Danil-Andrianov force-pushed the Issue-3421_AddingUIASupportForToolStripContainer branch from e2fcb01 to 64ab5e5 Compare January 31, 2022 07:34
@Danil-Andrianov Danil-Andrianov force-pushed the Issue-3421_AddingUIASupportForToolStripContainer branch from 64ab5e5 to c50eb44 Compare January 31, 2022 08:43
Copy link
Contributor

@vladimir-krestov vladimir-krestov left a comment

Choose a reason for hiding this comment

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

Looks good if it works

@Danil-Andrianov Danil-Andrianov force-pushed the Issue-3421_AddingUIASupportForToolStripContainer branch 2 times, most recently from 031f779 to 49fc605 Compare February 1, 2022 11:26
dreddy-work
dreddy-work previously approved these changes Feb 1, 2022
Copy link
Member

@dreddy-work dreddy-work left a comment

Choose a reason for hiding this comment

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

LGTM.

@SergeySmirnov-Akvelon
Copy link
Contributor

LGTM. Please take a look at some minor points.

@Danil-Andrianov Danil-Andrianov force-pushed the Issue-3421_AddingUIASupportForToolStripContainer branch 2 times, most recently from 679c004 to 3ceacca Compare February 3, 2022 05:21
@Danil-Andrianov Danil-Andrianov force-pushed the Issue-3421_AddingUIASupportForToolStripContainer branch from 3ceacca to 5d3d98c Compare February 3, 2022 06:16
@dreddy-work dreddy-work added the ready-to-merge PRs that are ready to merge but worth notifying the internal team. label Feb 10, 2022
@RussKie RussKie merged commit b56a086 into dotnet:main Feb 12, 2022
@ghost ghost added this to the 7.0 Preview2 milestone Feb 12, 2022
@Cassie-Li01
Copy link

Verified with .NET SDK 7.0.100-preview.3.22124.15 build, this issue is fixed
image
image

@ghost
Copy link

ghost commented Feb 25, 2022

Hi @Cassie-Li01, it looks like you just commented on a closed PR. The team will most probably miss it.
If you have a question - consider opening a new discussion thread. Alternatively, you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context.

@John-Qiao
Copy link
Contributor

Verified with .NET SDK 7.0.100-preview.2.22152.5 build, this issue is fixed as same as above test result.

@ghost
Copy link

ghost commented Mar 7, 2022

Hi @John-Qiao, it looks like you just commented on a closed PR. The team will most probably miss it.
If you have a question - consider opening a new discussion thread. Alternatively, you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context.

@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

ready-to-merge PRs that are ready to merge but worth notifying the internal team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants