Skip to content

Conversation

@GuY8528
Copy link
Contributor

@GuY8528 GuY8528 commented Dec 28, 2024

PRの概要

具体的な変更内容

Kapture.2025-01-23.at.18.01.10.mp4

影響範囲

動作要件

補足

レビューリクエストを出す前にチェック!

  • 改めてセルフレビューしたか
  • 手動での動作検証を行ったか
  • server の機能追加ならば、テストを書いたか
    • 理由: 書いた | server の機能追加ではない
  • 間違った使い方が存在するならば、それのドキュメントをコメントで書いたか
    • 理由: 書いた | 間違った使い方は存在しない
  • わかりやすいPRになっているか

@nakaterm
Copy link
Contributor

ちょっと挙動がおかしい

Kapture.2025-01-23.at.17.15.27.mp4

@nakaterm nakaterm marked this pull request as ready for review January 23, 2025 08:24
@nakaterm nakaterm marked this pull request as draft January 23, 2025 08:26
@nakaterm nakaterm changed the title Fix/card design カードの裏面削除・デザイン修正 Jan 23, 2025
Copy link
Contributor

@KaichiManabe KaichiManabe left a comment

Choose a reason for hiding this comment

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

LGTM
あざす!

@KaichiManabe KaichiManabe merged commit e4d8993 into main Jan 23, 2025
8 checks passed
@KaichiManabe KaichiManabe deleted the fix/Card-design branch January 23, 2025 09:35
KaichiManabe added a commit that referenced this pull request Jan 23, 2025
# PRの概要

- 全員にいいねを送りました!をいい感じにスタイリングした (close #462)

## Note

> [!warning]
> #564 に依存

## 具体的な変更内容

<img width="315" alt="image"
src="https://github.com/user-attachments/assets/07b9e17d-5348-4da2-b246-b2e2ec2ace49"
/>

## レビューリクエストを出す前にチェック!

- [ ] 改めてセルフレビューしたか
- [ ] 手動での動作検証を行ったか
- [ ] server の機能追加ならば、テストを書いたか
  - 理由: 書いた | server の機能追加ではない
- [ ] 間違った使い方が存在するならば、それのドキュメントをコメントで書いたか
  - 理由: 書いた | 間違った使い方は存在しない
- [ ] わかりやすいPRになっているか

<!-- レビューリクエスト後は、Slackでもメンションしてお願いすることを推奨します。 -->

---------

Co-authored-by: GuY8528 <[email protected]>
Co-authored-by: KaichiManabe <[email protected]>
nakaterm added a commit that referenced this pull request Mar 6, 2025
# PRの概要

* カードの裏面を削除し、詳細表示 (#565) が出るようにした (close #493)
* カード表面のデザインを修正した。

## 具体的な変更内容


https://github.com/user-attachments/assets/ce3e3752-bc73-46d8-8b93-103be2b9ccf4


## 影響範囲

## 動作要件

## 補足

## レビューリクエストを出す前にチェック!

- [ ] 改めてセルフレビューしたか
- [ ] 手動での動作検証を行ったか
- [ ] server の機能追加ならば、テストを書いたか
  - 理由: 書いた | server の機能追加ではない
- [ ] 間違った使い方が存在するならば、それのドキュメントをコメントで書いたか
  - 理由: 書いた | 間違った使い方は存在しない
- [ ] わかりやすいPRになっているか

<!-- レビューリクエスト後は、Slackでもメンションしてお願いすることを推奨します。 -->

---------

Co-authored-by: naka-12 <[email protected]>
nakaterm added a commit that referenced this pull request Mar 6, 2025
# PRの概要

- 全員にいいねを送りました!をいい感じにスタイリングした (close #462)

## Note

> [!warning]
> #564 に依存

## 具体的な変更内容

<img width="315" alt="image"
src="https://github.com/user-attachments/assets/07b9e17d-5348-4da2-b246-b2e2ec2ace49"
/>

## レビューリクエストを出す前にチェック!

- [ ] 改めてセルフレビューしたか
- [ ] 手動での動作検証を行ったか
- [ ] server の機能追加ならば、テストを書いたか
  - 理由: 書いた | server の機能追加ではない
- [ ] 間違った使い方が存在するならば、それのドキュメントをコメントで書いたか
  - 理由: 書いた | 間違った使い方は存在しない
- [ ] わかりやすいPRになっているか

<!-- レビューリクエスト後は、Slackでもメンションしてお願いすることを推奨します。 -->

---------

Co-authored-by: GuY8528 <[email protected]>
Co-authored-by: KaichiManabe <[email protected]>
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.

カードの回転機能と裏面を削除する 締切: 12/28-31

4 participants