Skip to content
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

chore: update navigation structure, wip #85

Merged
merged 14 commits into from
Sep 17, 2024
Merged

chore: update navigation structure, wip #85

merged 14 commits into from
Sep 17, 2024

Conversation

tiankaima
Copy link
Member

No description provided.

@tiankaima tiankaima force-pushed the dev/tkm-wip branch 2 times, most recently from 03a6a4c to e4b5c12 Compare September 12, 2024 07:25
@tiankaima tiankaima marked this pull request as ready for review September 12, 2024 09:42
@tiankaima tiankaima requested a review from taoky September 12, 2024 09:48
@tiankaima tiankaima merged commit 39e1d5c into master Sep 17, 2024
1 check passed
github-actions bot pushed a commit that referenced this pull request Sep 17, 2024
Tiankai Ma: [39e1d5c] Merge pull request #85 from ustclug/dev/tkm-wip
@tiankaima tiankaima deleted the dev/tkm-wip branch September 17, 2024 06:20
>
> 目前 Jekyll 的配置支持使用「直接指向 `.md` 文件链接 (相对目录)」的写法,但是并未全站采用这样的做法,理由如下:
>
> 1. 目录与文件名重名的问题,例如之前 `_wiki/` 目录下同时存在 `./scripts.md` 和 `./scripts/`。现将 `./scripts.md` 调整到 `./scripts/index.md`,并在其中配置 `permalink: /wiki/scripts/` 以保持原有链接的有效性。
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dir/index.md 会自动渲染为 dir/index.html 并且具有 dir/ 的固定链接,为文件名为 index.md 的文件加 permalink: 是没有必要的

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

图片

bundle exec jekyll serve 不加 permalink 就有点问题, 但是 index.html 确实在那里就是了...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(顺带一提这个写法也是我抄的就是了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants