Add governance documentation and sidebar graph icon integration#8045
Add governance documentation and sidebar graph icon integration#8045Deven14125 wants to merge 7 commits intowebpack:mainfrom
Conversation
Add several Governance docs under src/content/contribute (Overview, Charter, AI Policy, Member Expectations, Moderation Policy, Working Groups) and a new graph SVG icon. Integrate the graph icon into SidebarItem (import icon, use as toggle for Dependency Graph and add static state), update related SCSS for static/graph toggle styles, and make a small ESLint comment spacing tweak in Sidebar.jsx.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Review This My PR |
| ## Reference | ||
|
|
||
| - [Simdutf](https://github.com/simdutf/simdutf/blob/master/AI_USAGE_POLICY.md) | ||
| - [LLVM AI Tool Use Policy](https://discourse.llvm.org/t/rfc-llvm-ai-tool-policy-human-in-the-loop/89159) No newline at end of file |
There was a problem hiding this comment.
Remove these files from PR
There was a problem hiding this comment.
Pull request overview
Adds governance-related documentation pages under the Contribute section and introduces a new “graph” SVG icon intended for sidebar display (notably around the “Dependency Graph” concept).
Changes:
- Added multiple Governance MDX pages (overview, charter, working groups, moderation policy, member expectations, AI policy).
- Added a new
graph.svgicon and updatedSidebarItemto render it in the sidebar. - Updated sidebar styling/behavior (toggle modifiers + rotation selector) and tweaked an ESLint disable comment.
Reviewed changes
Copilot reviewed 8 out of 10 changed files in this pull request and generated 19 comments.
Show a summary per file
| File | Description |
|---|---|
| src/styles/icons/graph.svg | New graph icon used in the sidebar. |
| src/content/contribute/Governance-working-groups.mdx | New working-groups governance page content. |
| src/content/contribute/Governance-Overview.mdx | New governance overview page content. |
| src/content/contribute/Governance-moderation-policy.mdx | New moderation policy page content. |
| src/content/contribute/Governance-member-expectations.mdx | New leadership/member expectations page content. |
| src/content/contribute/Governance-charter.mdx | New TSC charter page content. |
| src/content/contribute/Governance-ai-policy.mdx | New AI usage policy page content. |
| src/components/SidebarItem/SidebarItem.scss | Adds toggle modifiers and changes which toggle selector rotates when open. |
| src/components/SidebarItem/SidebarItem.jsx | Switches toggle icon rendering to GraphIcon and adds a static graph icon case. |
| src/components/Sidebar/Sidebar.jsx | Adjusts an ESLint directive comment formatting. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| {anchors.length > 0 ? ( | ||
| <ChevronRightIcon | ||
| <GraphIcon | ||
| width={15} | ||
| height={17} | ||
| fill="#175d96" |
| } | ||
|
|
||
| .sidebar-item__toggle { | ||
| .sidebar-item__toggle--graph { |
| {pages.map((page, index) => { | ||
| const displayGroup = group !== page.group && page.group !== "-"; | ||
| // eslint-disable-next-line react-hooks/immutability | ||
| // eslint-disable-next-line react-hooks/immutability |
There was a problem hiding this comment.
ok , I Fixed it.
There was a problem hiding this comment.
Hello, I fixed it
| --- | ||
| title: Working Groups | ||
| group: Contribute | ||
| sort: 4 |
| that working group. | ||
|
|
||
| You can submit the working group charter for ratification by sending | ||
| a pull request to this document to add the charter it to the |
| --- | ||
| title: Moderation Policy | ||
| group: Contribute | ||
| sort: 3 |
| * TSC members may Remove or Block an individual from the webpack | ||
| GitHub Organizations. | ||
| * For any Removal or Blocking action, an issue describing the reasons for the | ||
| action, and identifying the Github account being acted upon, must be posted |
|
|
||
| --- | ||
|
|
||
| _This document is an adaption of the Node.js project [Moderation Policy](https://github.com/nodejs/admin/blob/HEAD/Moderation-Policy.md)_ |
| _This document is an adaption of the Node.js project [Moderation Policy](https://github.com/nodejs/admin/blob/HEAD/Moderation-Policy.md)_ | ||
|
|
||
| [Code of Conduct]: https://github.com/webpack/webpack/blob/main/CODE_OF_CONDUCT.md | ||
| [webpack Technical Steering Committee]: ./CHARTER.md |
| @@ -0,0 +1,64 @@ | |||
| --- | |||
| title: Ai Policy | |||
Replace the chevron icon with a graph icon for the sidebar item and add a static graph icon for the "Dependency Graph" entry when it has no anchors. Import a new SVG (src/styles/icons/graph.svg), add an isDependencyGraph check, and adjust classNames to apply a rotated graph toggle and a static variant. Also add SCSS rules for the static state (cursor default, hover color) and rename the rotated toggle selector. Minor whitespace fix in an eslint disable comment in Sidebar.jsx.
…bpack.js.org into fix/content-modify
|
Now Review, if any then tell me. |
|
|
||
| _This document is an adaption of the [Node.js project Governance Model](https://github.com/nodejs/node/blob/main/GOVERNANCE.md) and the [ESlint project Governance Model](https://github.com/eslint/eslint/blob/main/docs/src/contribute/governance.md)_ | ||
|
|
||
| [TSC Charter]: https://github.com/nodejs/TSC/blob/HEAD/TSC-Charter.md |
There was a problem hiding this comment.
Please remove this files from your PR, they are auto generated
Delete multiple governance-related MDX files from src/content/contribute: Governance-Overview.mdx, Governance-ai-policy.mdx, Governance-charter.mdx, Governance-member-expectations.mdx, Governance-moderation-policy.mdx, and Governance-working-groups.mdx. These pages referenced the webpack/governance sources and have been removed from the site content.
|
Review Now |
Adjust the indentation of an `// eslint-disable-next-line react-hooks/immutability` comment in src/components/Sidebar/Sidebar.jsx to match the file's formatting. This is a whitespace-only change with no functional impact.
…bpack.js.org into fix/content-modify
|
I Fixed it. |
a0ca8d1 to
cb19bd1
Compare
|
/cc @evenstensberg do we need to improve this? |
|
@Deven14125 Thanks for your contribution but our navbar icons are good as they are. I'd try to find issues that exists in our issue list to make it more probable to get a pr merged. |
|
Thanks for the feedback! I’ll look into the issue list and work on something aligned with the current needs of the project. Appreciate the guidance! |
Add several Governance docs under src/content/contribute (Overview, Charter, AI Policy, Member Expectations, Moderation Policy, Working Groups) and a new graph SVG icon. Integrate the graph icon into SidebarItem (import icon, use as toggle for Dependency Graph and add static state), update related SCSS for static/graph toggle styles, and make a small ESLint comment spacing tweak in Sidebar.jsx.
Summary
What kind of change does this PR introduce?
Did you add tests for your changes?
Does this PR introduce a breaking change?
If relevant, what needs to be documented once your changes are merged or what have you already documented?
Use of AI