Skip to content
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

Port PropertyGridDesigner and NotifyIconDesigner to runtime #4908 #9645

Conversation

Epica3055
Copy link
Member

@Epica3055 Epica3055 commented Aug 3, 2023

Related to #4908

Proposed changes

  • Port PropertyGridDesigner and NotifyIconDesigner to runtime

Customer Impact

  • PropertyGridDesigner and NotifyIconDesigner can be designed in runtime

Regression?

  • No

Risk

  • Minimal

Screenshots

Before

004

After

003

Test methodology

  • Manually (added PropertyGridDesigner and NotifyIconDesigner to DemoConsole test app)

Test environment(s)

  • .Net 8.0.0-rc.1.23401.22
Microsoft Reviewers: Open in CodeFlow

@ghost ghost assigned Epica3055 Aug 3, 2023
@ghost ghost added the draft draft PR label Aug 3, 2023
@Epica3055 Epica3055 force-pushed the Issue_4908_Port_PropertyGridDesignerAndNotifyIconDesigner branch from 61b21a1 to eae980e Compare August 4, 2023 00:27
@Epica3055 Epica3055 removed the draft draft PR label Aug 4, 2023
@Epica3055 Epica3055 marked this pull request as ready for review August 4, 2023 01:02
@Epica3055 Epica3055 requested a review from a team as a code owner August 4, 2023 01:02
LeafShi1
LeafShi1 previously approved these changes Aug 4, 2023
@Tanya-Solyanik Tanya-Solyanik added this to the .NET 9.0 milestone Aug 4, 2023
Copy link
Member

@Tanya-Solyanik Tanya-Solyanik 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, but the new functionality should go into the feature/9.0 branch because we want to stabilize the main branch. Could you please rebase your PR?

@Tanya-Solyanik Tanya-Solyanik added the 📭 waiting-author-feedback The team requires more information from the author label Aug 4, 2023
@Epica3055 Epica3055 changed the base branch from main to feature/9.0 August 7, 2023 01:46
@Epica3055 Epica3055 dismissed LeafShi1’s stale review August 7, 2023 01:46

The base branch was changed.

@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Aug 7, 2023
@dreddy-work dreddy-work added the 📭 waiting-author-feedback The team requires more information from the author label Aug 7, 2023
@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Aug 8, 2023
@dreddy-work dreddy-work added the 📭 waiting-author-feedback The team requires more information from the author label Aug 8, 2023
@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Aug 9, 2023
@Tanya-Solyanik Tanya-Solyanik added the 📭 waiting-author-feedback The team requires more information from the author label Aug 10, 2023
@Epica3055 Epica3055 force-pushed the Issue_4908_Port_PropertyGridDesignerAndNotifyIconDesigner branch from 1700db5 to 251491b Compare August 10, 2023 02:47
@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Aug 10, 2023
@Tanya-Solyanik
Copy link
Member

@Epica3055, @LeafShi1 - please coordinate the fix for nullability definitions that blocks your PRs.

Tanya-Solyanik
Tanya-Solyanik previously approved these changes Aug 10, 2023
@Tanya-Solyanik Tanya-Solyanik added the 📭 waiting-author-feedback The team requires more information from the author label Aug 10, 2023
@Tanya-Solyanik Tanya-Solyanik added ready-to-merge PRs that are ready to merge but worth notifying the internal team. and removed 📭 waiting-author-feedback The team requires more information from the author labels Aug 10, 2023
@Tanya-Solyanik Tanya-Solyanik merged commit 3e9f4f4 into dotnet:feature/9.0 Aug 11, 2023
9 checks passed
@ghost ghost removed the ready-to-merge PRs that are ready to merge but worth notifying the internal team. label Aug 11, 2023
@MelonWang1
Copy link
Contributor

Verified this PR in feature/9.0 branch, now PropertyGridDesigner and NotifyIconDesigner is supported in runtime, we can open NotifyIcon smarttag and modify PropertyGrid in DemoConsole app by its some properties:

9645.mp4

@ghost ghost locked as resolved and limited conversation to collaborators Sep 10, 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.

None yet

5 participants