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

タイトルが2024になってしまっているのを修正 #11

Merged
merged 6 commits into from
Nov 13, 2024

Conversation

PharaohKJ
Copy link
Contributor

いろいろと参照で2024のをみちゃってるんで、次スポンサーやら登壇者やらやるときは、旧レイアウトに2024と付けて参照を付け替える作業がいる。

とりあえず今回はトップページだけは対応できるようした。

@PharaohKJ PharaohKJ self-assigned this Nov 1, 2024
@PharaohKJ
Copy link
Contributor Author

問題のあったファイル src/layouts/Vanilla.astrosrc/layouts/Vanilla2024.astro にコピーして、2024のページはそっちみるように。2025は src/layouts/Vanilla.astro の中身を2025にして対応。

ページタイトルから年度を削除
@PharaohKJ
Copy link
Contributor Author

タイトルから年度表記を消した。なので Vanilla2024.astro はなくなった。
しかし、登壇者(Speaker)とスポンサー(Sponsor)のastroファイルにTOPページへのリンクが / とあり、これを /2024/ と置き換えた。置き換えると使い回せないので、2024年用のファイルとして保存した。あんまりよくなさそうだけどとりあえずこれで。

Copy link
Member

@xin9le xin9le left a comment

Choose a reason for hiding this comment

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

以下のフォルダについて、過去分はディレクトリを切ると後々の管理が楽になると思います。翌年に 2026 のディレクトリを切るだけでよくなるので。astro 詳しくないからできるかは不明ですが、できるのであれば。

  • components
  • layouts
- components
    - 2024
        - Hero.tsx
    - 2025
        - Hero.tsx
- layouts
    - Layout.astro
    - Vanilla.astro
    - 2024
        - Speaker.astro
        - Sponser.astro
    - 2025
        - Speaker.astro
        - Sponsor.astro

@PharaohKJ
Copy link
Contributor Author

ディレクトリ区切り。あーなるほど。絶対そのほうがいいね。

@PharaohKJ PharaohKJ requested a review from xin9le November 3, 2024 00:49
@PharaohKJ
Copy link
Contributor Author

できたけど、これやるならタイトルに西暦入れてもいいかもしんないね。楽だし。

Copy link
Member

@xin9le xin9le left a comment

Choose a reason for hiding this comment

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

image

Copy link
Contributor

@hikaruworld hikaruworld left a comment

Choose a reason for hiding this comment

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

[IMO] | が取り残されていたので(本当はtitle入るのが期待値何ですが)、一旦 | も外してもらえるとありがたいです。
(面倒そうなら一旦入れます!)

スクリーンショット 2024-11-12 11 40 31

@PharaohKJ
Copy link
Contributor Author

@hikaruworld @xin9le
マージしてだしちまいます。問題あったらまた更新で。

@PharaohKJ PharaohKJ merged commit 61fc164 into main Nov 13, 2024
@PharaohKJ PharaohKJ deleted the fix-title-of-toppage branch November 13, 2024 05:21
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.

3 participants