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

パンくずリストを設置 #139

Merged
merged 3 commits into from
Jun 4, 2024
Merged

パンくずリストを設置 #139

merged 3 commits into from
Jun 4, 2024

Conversation

mehm8128
Copy link
Contributor

@mehm8128 mehm8128 commented Jun 2, 2024

close #127

@mehm8128 mehm8128 requested a review from Pugma June 2, 2024 08:55
@mehm8128 mehm8128 self-assigned this Jun 2, 2024
Copy link
Collaborator

@Pugma Pugma left a comment

Choose a reason for hiding this comment

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

1点確認をお願いします:pray:

全体的に、stringでURLを記述するよりも
{ name: 'user', params: { userId: 123 }}
みたいに書いたほうが良さそうな気がしたんですが、型を制御するのが難しかった感じですかね…?

v-if="index !== paths.length - 1"
:to="path.link"
:class="$style.link"
:aria-current="route.path === path.link ? 'page' : undefined"
Copy link
Collaborator

Choose a reason for hiding this comment

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

ここなんですが、常にfalseの値を取るような気がします

Copy link
Contributor Author

Choose a reason for hiding this comment

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

夜直します🙏

@mehm8128
Copy link
Contributor Author

mehm8128 commented Jun 3, 2024

リンクについてなのですが、多分そっちの方がいいけど今dashboardも含めて基本stringで扱ってるので、やるなら別で一気に変えるのがよさそうです

@mehm8128 mehm8128 requested a review from Pugma June 3, 2024 12:11
Copy link
Collaborator

@Pugma Pugma left a comment

Choose a reason for hiding this comment

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

よさそうです!
修正対応と、navタグへの置換もありがとうございました

@mehm8128 mehm8128 merged commit 1213fcb into master Jun 4, 2024
8 checks passed
@mehm8128 mehm8128 deleted the feat/breadcrumb branch June 4, 2024 01:30
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