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

ブックマークが無いときの表示を変更 #3335

Merged
merged 2 commits into from
Oct 1, 2021

Conversation

Ichiyo14
Copy link
Contributor

ref #3023

ブックマークが0件のときの表示を設定する。
#3307 に習い泣き顔を表示する。

修正前

スクリーンショット 2021-10-01 7 21 45

## 修正後

スクリーンショット 2021-10-01 8 18 07

@Ichiyo14
Copy link
Contributor Author

Ichiyo14 commented Oct 1, 2021

@kaiyu-tech
お疲れ様です。お時間ある時にレビューをよろしくお願いします!

Copy link
Contributor

@kaiyu-tech kaiyu-tech left a comment

Choose a reason for hiding this comment

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

@Ichiyo14
動作確認しました!
一点、コメントしました。ご確認お願いします🙏

Comment on lines 22 to 25
.o-empty-message__icon
i.far.fa-sad-tear
p.o-empty-message__text
| ブックマークしているものはありません。
Copy link
Contributor

@kaiyu-tech kaiyu-tech Oct 1, 2021

Choose a reason for hiding this comment

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

動作確認してみましたが、以下のように表示されました。
image

以下のコードの部分のインデントが間違っているために、o-empty-message__textに対してo-empty-message__iconのスタイルが当たってしまっているようです。

Suggested change
.o-empty-message__icon
i.far.fa-sad-tear
p.o-empty-message__text
| ブックマークしているものはありません。
.o-empty-message__icon
i.far.fa-sad-tear
p.o-empty-message__text
| ブックマークしているものはありません。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

slimで記述をするときにはインデントが重要なんですね!
理解できました😊

Copy link
Contributor

Choose a reason for hiding this comment

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

はい、そのようですね!
私も知りませんでした😁

@Ichiyo14
Copy link
Contributor Author

Ichiyo14 commented Oct 1, 2021

@kaiyu-tech
ご指摘いただいた箇所の修正をしました。ご確認よろしくお願いします🙏

@kaiyu-tech
Copy link
Contributor

@Ichiyo14
確認しました!LGTMです🚀

@Ichiyo14
Copy link
Contributor Author

Ichiyo14 commented Oct 1, 2021

@kaiyu-tech
迅速なレビュー対応ありがとうございました!

@komagata
レビューをよろしくお願いします🙏

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認しました、OKですー🙆‍♂️

@komagata komagata merged commit c3678be into main Oct 1, 2021
@komagata komagata deleted the feature/display-when-there-are-zero-bookmarks branch October 1, 2021 14:11
@github-actions github-actions bot mentioned this pull request Oct 1, 2021
14 tasks
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