Merged
Conversation
Deploying sapling with
|
| Latest commit: |
f8ea0c6
|
| Status: | ⚡️ Build in progress... |
There was a problem hiding this comment.
Pull request overview
This PR implements an interactive file tree explorer component to replace static text-based directory structures throughout the documentation. The implementation is inspired by PaperMC's documentation and provides a more user-friendly way to visualize file and folder hierarchies with hover-over descriptions.
Key changes:
- Added a reusable
StructureExplorerReact component with popup descriptions - Created 10 data structure files covering various Minecraft server configurations
- Converted multiple
.mdfiles to.mdxto support React components - Added
@iconify/reactdependency for folder/file icons - Implemented dark/light theme support for the component
Reviewed changes
Copilot reviewed 28 out of 30 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.json | Added baseUrl configuration to support path aliases for imports |
| src/types/ExplorerNode.ts | Defined TypeScript interface for file tree node structure |
| src/data/structures/*.ts | Created 10 data structure files (viaproxy, server directories, BDS, Nukkit, Maven, Gradle, etc.) |
| src/data/structures/index.ts | Barrel export file for all structure data |
| src/css/structure_explorer.css | Component styling with tree lines, hover effects, and popup windows |
| src/css/base/theme.css | Added CSS variables for dark/light theme support |
| src/components/StructureExplorer.tsx | Main React component implementing the interactive file tree |
| package.json | Added @iconify/react dependency |
| pnpm-lock.yaml | Lock file updates for new dependency |
| docs/*.mdx | Converted 7 documentation files from .md to .mdx and integrated the component |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)
docs-java/start/basic/server-dir-structure.mdx:7
- The import statement uses a direct path to the data file instead of importing from the barrel export in the index. This should use
import { serverDirStructureData } from "@site/src/data/structures"to maintain consistency with other imports and make refactoring easier.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Dreeam-qwq
added
the
type: docs site
wling-art
force-pushed
the
feat/filetree2
branch
from
f8ea0c6 to
5109261
Compare
wling-art
force-pushed
the
feat/filetree2
branch
from
f7737ae to
f8ea0c6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
仿照 https://github.com/PaperMC/docs/blob/5fef1d92b5d7a4744800cfdda2a13ce596451022/src/components/config/ConfigurationStructureDiagram.tsx
示例图：
