Skip to content

style: fix EventCard layout#309

Merged
PaiJi merged 1 commit into
mainfrom
style/fix-event-card-height
May 16, 2026
Merged

style: fix EventCard layout#309
PaiJi merged 1 commit into
mainfrom
style/fix-event-card-height

Conversation

@PaiJi

@PaiJi PaiJi commented May 16, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

发布说明

  • 样式优化
    • 优化了事件卡片图片显示的布局和响应式设计,提升了图片在不同屏幕尺寸下的展示效果。

Review Change Stack

@PaiJi PaiJi merged commit 5605031 into main May 16, 2026
4 checks passed
@PaiJi PaiJi deleted the style/fix-event-card-height branch May 16, 2026 18:59
@coderabbitai

coderabbitai Bot commented May 16, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 53293a24-749f-4264-bb62-e9f5307fcfbf

📥 Commits

Reviewing files that changed from the base of the PR and between 6be0cbb and 3482e33.

📒 Files selected for processing (1)
  • src/components/eventCard/index.tsx

📝 Walkthrough

概览

EventCard 组件的封面区域进行了布局样式调整:外层容器重新定义了响应式高度约束,EventCover 的前景渲染层新增了溢出隐藏与绝对定位的居中结构,并扩展了前景图片的宽度显示范围。

变更

事件卡片封面布局优化

图层 / 文件 摘要
封面容器与图片宽度约束调整
src/components/eventCard/index.tsx
EventCard 外层容器的 className 移除了 md:h-full 并引入 min-h-[150px] md:h-full 约束;EventCover 重构前景区域,增加 self-stretch/overflow-hidden 样式,新增绝对定位的居中覆盖层,调整 Image 的 containerClassName 和前景图 className 以实现全尺寸显示(从 h-full 扩展为 h-full w-full)。

代码审查工作量估计

🎯 2 (简单) | ⏱️ ~12 分钟

兔子的庆祝诗

卡片封面焕新妆,布局柔和又敞亮,
约束条件精心装,响应式设计更向往,
宽度撑满全屏广,视觉层级藏其中,✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch style/fix-event-card-height

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.

@github-actions

Copy link
Copy Markdown

📦 Next.js Bundle Analysis for furrycons-site

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

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.

1 participant