-
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
High-priority list of properties requiring attention (3) #6989
Comments
@jjmew Here's the issue tracking the property page tweaks that I think someone other than me or @drewnoakes could make. I've excluded items that I view as low priority (e.g., changes specifically to support non-SDK projects) or are complex. @drewnoakes I split these out of #6907. Let me know if there are other things we could include here. |
@tmeschter @jjmew I think it would be easier to track if these were broken out into separate issues. Priorities can be fluid, and these may be assigned to different people. |
@drewnoakes Per @jjmew's request, I've kept these in a single issue. |
@drewnoakes the reason was because I could assign it directly to @MiYanni |
All of these have either been addressed or split into other issues. |
This work item has been split off of #6907 to contain the highest priority items.
The following properties are currently included in the UI and require some sort of additional treatment to make them work correctly.
Nullable
should only be visible when using C# 8 or greaterNullable
on that.Opened Determine general solution for properties that have no initial value #7360Nullable
appearing with nothing selected<Nullable>
property. We don't currently have a good general solution to this, but it could be handled with anIInterceptingPropertyValueProvider
: if the property isn't set it could returndisabled
for the value.TargetFramework
andTargetFrameworks
properties on those properties actually being defined. That way only one or the other will show.TreatWarningsAsErrors
,NoWarn
,WarningsAsErrors
andWarningsNotAsErrors
(see Add UI for WarningsNotAsErrors #2409 for some context)The text was updated successfully, but these errors were encountered: