Skip to content

Conversation

@machida
Copy link
Member

@machida machida commented Dec 3, 2025

No description provided.

@coderabbitai
Copy link

coderabbitai bot commented Dec 3, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sass-2-css

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

machida and others added 25 commits December 18, 2025 00:43
.reactions__item-emojiにfont-size、line-height、vertical-alignを追加して、
絵文字のサイズと配置を制限しました。これにより、絵文字が親要素から
はみ出してcard-main-actionsのボタンの上に重なる問題を解決しました。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
Sassのネスト構文(&)が残っていたため、すべて標準的なCSSセレクタに展開しました。
これにより、ブラウザでの互換性が向上し、リアクション絵文字がボタンを
妨害する問題を解決します。

変更内容:
- .card-body + & → .card-body + .reactions
- &.is-reacted → .reactions__item.is-reacted
- &:hover → .reactions__item:hover
- &::after → .reactions__item-login-names::after
- その他すべてのネストセレクタを展開

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
.reactionsにz-index: 0を、.card-footerにposition: relativeとz-index: 1を追加して、
card-footer内のボタンがリアクション絵文字によってクリックを妨害されないようにしました。

また、card-footerのネスト構文も標準CSSに展開しました。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
machida and others added 2 commits December 18, 2025 01:27
.card-footerからposition: relativeとz-indexを削除し、
代わりに.card-main-actions__itemsにz-index: 1を追加しました。
これにより、リアクションアイテムがクリック可能になりながら、
card-main-actionsのボタンもリアクション絵文字に妨害されません。

また、card-footer.cssのネスト構文を標準CSSに展開しました。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
.reactionsのz-index: 0を削除し、代わりに.reactions__itemに
z-index: 2を追加しました。これにより、リアクションアイテムが
card-main-actions__items (z-index: 1)より上に配置され、
両方が正しくクリックできるようになります。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
machida and others added 5 commits December 18, 2025 12:11
.reactions__itemではなく親要素の.reactionsにz-index: 2を設定することで、
リアクション全体がcard-main-actions__items (z-index: 1)より上に配置されます。
これにより、リアクションアイテムが正しくクリックできるようになります。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
.reactions__item-emojiにpointer-events: noneを設定することで、
絵文字がクリックイベントを妨害せず、下の要素(card-main-actionsのボタン)が
正しくクリックできるようになります。

また、.reactionsのz-indexを削除してシンプルな解決策にしました。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
.reactions__itemにz-index: 2を追加することで、card-main-actions__items (z-index: 1)
より上に配置され、リアクションアイテムが正しくクリックできるようになります。

絵文字にはpointer-events: noneが設定されているため、絵文字がcard-main-actionsの
ボタンを妨害することはありません。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
- Set card-footer z-index to 1 to appear above reactions
- Increase reaction dropdown z-index to 10 to appear above card-footer
- Remove incorrect z-index from card-main-actions__items
- Add position relative to reactions__dropdown
- Update test to specifically target selected reactions to avoid ambiguity
@machida machida self-assigned this Dec 18, 2025
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