-
Notifications
You must be signed in to change notification settings - Fork 152
Initial draft of diagnostic redesign #3145
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
Conversation
@IEvangelist This is ready. I may need to adjust the category names and any invalid wording I'm using due to lack of knowledge. |
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 like it a lot, well done! I do want @davidfowl, @eerhardt, and @mitchdenny to see this and provide feedback.
I like splitting this out too. Can we confirm that the URLs given by the analyzers will still get people to the right place? |
They still work because they go to the overview article at a bookmark named after the ID. I kept the bookmarks, moving them to the table instead of a section heading. |
I think ideally, we'll update the |
Summary
Going to need help with wording.
Fixes #3143
Fixes #3142
Fixes #3141
Fixes #3140
Fixes #3139
Fixes #3131
Fixes #3126
Fixes #3057
Fixes #1658
TODO:
Internal previews