-
Notifications
You must be signed in to change notification settings - Fork 391
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
Reorder properties on Application page #7038
Reorder properties on Application page #7038
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm on the fence about whether Output Type or Target Framework should go first, but either is an improvement.
Yes I also contemplated this. Once we get telemetry around the frequency of property updates, we can revisit ordering using actual data to inform these decisions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
Historically in legacy projects, the Assembly Name and Default Namespace properties were manually kept in sync with the project by the user. With SDK-style projects, it's more common to go with the default behaviour of picking up the MSBuildProjectName property, which is computed from the project's file name. Given these properties are far less likely to be user edited, we move them further down the UI, allowing more commonly modified properties to rise to the top.
27e4a5e
to
4112be3
Compare
Historically in legacy projects, the Assembly Name and Default Namespace properties were manually kept in sync with the project by the user.
With SDK-style projects, it's more common to go with the default behaviour of picking up the
MSBuildProjectName
property, which is computed from the project's file name.Given these properties are far less likely to be user edited, we move them further down the UI, allowing more commonly modified properties to rise to the top.
Microsoft Reviewers: Open in CodeFlow