-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Diagram of VS/SDK component interactions #13127
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,173 @@ | ||||||||||||||||||||||||||||||||||||||||||
| <!DOCTYPE html> | ||||||||||||||||||||||||||||||||||||||||||
| <html lang="en"> | ||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||
| <head> | ||||||||||||||||||||||||||||||||||||||||||
| <meta charset="UTF-8"> | ||||||||||||||||||||||||||||||||||||||||||
| <meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||||||||||||||||||||||||||||||||||||||||||
| <title>Interactive Layer Diagram</title> | ||||||||||||||||||||||||||||||||||||||||||
| <style> | ||||||||||||||||||||||||||||||||||||||||||
| :root { | ||||||||||||||||||||||||||||||||||||||||||
| --highlight-color: black; | ||||||||||||||||||||||||||||||||||||||||||
| --vs-color: chocolate; | ||||||||||||||||||||||||||||||||||||||||||
| --sdk-color: #512bd4; | ||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||
| body { | ||||||||||||||||||||||||||||||||||||||||||
| font-family: system-ui, -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen, Ubuntu, Cantarell, 'Open Sans', 'Helvetica Neue', sans-serif, sans-serif; | ||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||
| font-family: system-ui, -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen, Ubuntu, Cantarell, 'Open Sans', 'Helvetica Neue', sans-serif, sans-serif; | |
| font-family: system-ui, -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen, Ubuntu, Cantarell, 'Open Sans', 'Helvetica Neue', sans-serif; |
Copilot
AI
Jan 27, 2026
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.
.component { outline: none; } is effectively overridden by #diagram div { outline: ... } due to selector specificity, so the outline:none doesn’t take effect. Either remove the redundant outline:none or narrow #diagram div so it only targets the elements you intend to outline (e.g., components vs. container/header divs).
Copilot
AI
Jan 27, 2026
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.
This doc uses <tt> for inline code/filenames, but <tt> is obsolete in HTML5 and has inconsistent styling across browsers. Prefer <code> (optionally styled via CSS) for dotnet build, MSBuild.exe, filenames, etc.
| <label for="sdkbuild">Build with <tt>dotnet build</tt></label> | |
| <br> | |
| <input type="radio" name="buildOption" id="designtime" value="designtime"> | |
| <label for="designtime">VS project load/Design-time build</label> | |
| <br> | |
| <input type="radio" name="buildOption" id="vsbuild" value="vsbuild"> | |
| <label for="vsbuild">Build with <tt>MSBuild.exe</tt> or VS F5</label> | |
| <br> | |
| <input type="radio" name="buildOption" id="vsbuild10" value="vsbuild10"> | |
| <label for="vsbuild10">Build with <tt>MSBuild.exe</tt> or VS F5 (.NET 10+)</label><br> | |
| <label for="sdkbuild">Build with <code>dotnet build</code></label> | |
| <br> | |
| <input type="radio" name="buildOption" id="designtime" value="designtime"> | |
| <label for="designtime">VS project load/Design-time build</label> | |
| <br> | |
| <input type="radio" name="buildOption" id="vsbuild" value="vsbuild"> | |
| <label for="vsbuild">Build with <code>MSBuild.exe</code> or VS F5</label> | |
| <br> | |
| <input type="radio" name="buildOption" id="vsbuild10" value="vsbuild10"> | |
| <label for="vsbuild10">Build with <code>MSBuild.exe</code> or VS F5 (.NET 10+)</label><br> |
Copilot
AI
Jan 27, 2026
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.
This doc uses <tt> for inline code, which is obsolete in HTML5. Consider switching these to <code> for better semantics and accessibility.
Copilot
AI
Jan 27, 2026
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.
GitHub doesn’t render repository .html files as pages in the normal file view, so this diagram won’t be directly viewable without using “Raw”/downloading. Consider adding a short instruction near the top (how to view), and/or adding a rendered PNG/SVG + a .md wrapper so the diagram is visible in GitHub’s documentation flow (similar to other diagrams in documentation/*.md).
Copilot
AI
Jan 27, 2026
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.
<tt> is obsolete in HTML5. Prefer <code> for inline code like dotnet (and style it via CSS if you want a monospaced look).
Copilot
AI
Jan 27, 2026
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.
<tt> is obsolete in HTML5; these filenames would be better marked up with <code> for correct semantics and more consistent rendering.
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.
The page
<title>(“Interactive Layer Diagram”) doesn’t match the main heading (“MSBuild/.NET SDK/Visual Studio Interactions”). Aligning them makes the document easier to find/bookmark and reduces ambiguity in browser tabs/history.