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

リンクタグと感想について #51

Open
emi-h opened this issue Oct 2, 2022 · 1 comment
Open

リンクタグと感想について #51

emi-h opened this issue Oct 2, 2022 · 1 comment

Comments

@emi-h
Copy link

emi-h commented Oct 2, 2022

実装お疲れ様でした!

リンクタグについて

Next.jsのLinkコンポーネントを使用する方が良いのではないかと思いました。
リンク先のページ全体の再レンダリングを防げたり、SEO的にもベターだと思います。
Lighthouseでリンクをクロールできない(Links are not crawlable)とのメッセージが出ていますが、
それも解消できると思います。

Next.jsドキュメント:https://nextjs-ja-translation-docs.vercel.app/docs/api-reference/next/link

感想(今回の開発だけでなく全体的な話もあります)

・綺麗に実装されていると思いました。(ブログやポートフォリオの内容も濃い!)
・dynamicインポート機能を初めて知りました。今度使ってみたいと思います。
・ホバー時のスタイル(背景グレー)が見やすくて良いと思いました。
・開発経験が豊富でリポジトリが充実しているのもあると思いますが、
 見せたいリポジトリをgqlで指定して出力されているのが良いと思いました。
 私もポートフォリオを作るときは考え方を真似させてください!

@pitang1965
Copy link
Owner

next/linkは一部では使っています。
AnchorSelf (src/components/common/AnchorSelf.tsx) でラップし、7つのコンポーネントで使用しています。ただ、外部のサイトに対しては使っていません。そのことは問題ないでしょうか?

Lighthouseでリンクをクロールできない(Links are not crawlable)とのメッセージが出ている件、気づきませんでした。調べて対応したいと思います。

感想ありがとうございます。
見せたいリポジトリは、https://github.com/pitang1965 でもともとピンどめしているものを引っ張ってきています。

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

No branches or pull requests

2 participants