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

classの重複によるCSSバグを修正 #7480

Merged
merged 1 commit into from
Mar 9, 2024

Conversation

hirano-vm4
Copy link
Contributor

@hirano-vm4 hirano-vm4 commented Mar 1, 2024

Issue

概要

検索結果のラベルが二重になってしまっているバグの解消になります。現状では「定期イベント」「特別イベント」「プラクティス」以外のラベルがすべて二重の囲いになっています。

スクリーンショット 2024-03-02 20 28 01

image
このように同じCSSが当たってしまってる。

変更確認方法

  1. bug/fix-duplicate-search-labelsをローカルに取り込む
  2. ログイン(権限は一般ユーザーでもOK)
  3. 空欄のまま検索
スクリーンショット 2024-03-02 20 20 48 4. 「定期イベント」「特別イベント」「プラクティス」以外のラベルが二重になっていないことを確認

Screenshot

変更前

スクリーンショット 2024-03-02 20 33 57

変更後

スクリーンショット 2024-03-02 20 31 04

@hirano-vm4 hirano-vm4 self-assigned this Mar 2, 2024
@hirano-vm4 hirano-vm4 marked this pull request as ready for review March 2, 2024 11:36
@hirano-vm4 hirano-vm4 requested a review from mousu-a March 2, 2024 11:45
@hirano-vm4
Copy link
Contributor Author

hirano-vm4 commented Mar 2, 2024

@mousu-a

レビュー依頼

お世話になります!先日からチーム開発に参加しました🙏

お忙しい中、申し訳ありません。レビューをお願いすることは可能でしょうか?

内容はgood first issueの1PのIssueなのでシンプルなもので、大きく時間はかからない内容かと思います🙏
状況的に難しいようでしたら、気にせずにおっしゃってください!

ご検討のほど、よろしくお願いいたします🙇‍♂️

@mousu-a
Copy link
Contributor

mousu-a commented Mar 2, 2024

@hirano-vm4

レビュー依頼承知いたしましたー!
自分も先日入ったばかりなので一緒に頑張っていきましょー!
こちらこそこれからお世話になります🙇‍♂️
すぐお返ししますね〜😄

@hirano-vm4
Copy link
Contributor Author

@mousu-a

快諾してくださり助かります🙏 同じくらいのタイミングですね!同じチームとしてお互い相談しながらやっていきましょう🎉

無理ないタイミングで大丈夫ですので、よろしくお願いします🙏

@mousu-a
Copy link
Contributor

mousu-a commented Mar 2, 2024

@hirano-vm4
OKだと思いますー!Approveさせていただきました〜👍

@hirano-vm4
Copy link
Contributor Author

@mousu-a
夜遅くにありがとうございました🙏引き続きよろしくお願いいたします💪

@hirano-vm4
Copy link
Contributor Author

@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 074312a into main Mar 9, 2024
5 checks passed
@komagata komagata deleted the bug/fix-duplicate-search-labels branch March 9, 2024 13:36
@github-actions github-actions bot mentioned this pull request Mar 9, 2024
29 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