Skip to content

为掼蛋队伍计分行新增 .guandan-teams 包裹以修复移动端样式#9

Merged
TypeThe0ry merged 1 commit into
mainfrom
codex/fix-guandan-team-score-area-styling
Jun 27, 2026
Merged

为掼蛋队伍计分行新增 .guandan-teams 包裹以修复移动端样式#9
TypeThe0ry merged 1 commit into
mainfrom
codex/fix-guandan-team-score-area-styling

Conversation

@TypeThe0ry

Copy link
Copy Markdown
Owner

Motivation

  • 移动端媒体查询在 static/css/style.css 的两处(@media (max-width: 768px)@media (max-width: 480px))都针对 .guandan-teams 设定了网格和隐藏规则,但模板中缺少该容器导致样式未生效。
  • 目标是让现有响应式样式在小屏设备上一致生效,同时保持现有 Vue v-for 渲染逻辑不变。

Description

  • static/index.html 的掼蛋计分板处将多处 .guandan-team-row 包裹进新增的容器 <div class="guandan-teams">
  • 保持 v-for="team in guandanTeams" 仍然在每个 .guandan-team-row 元素上,保证数据渲染行为不变。
  • 未修改 static/css/style.css,并确认两处移动端媒体查询中关于 .guandan-teams 的规则在文件中一致存在。

Testing

  • 运行 git diff --check 检查代码风格/冲突,结果通过。
  • 使用 rg -n "guandan-teams|guandan-team-row" static/index.html static/css/style.css 验证模板与样式定位正确,结果显示新增容器和两处媒体查询均已匹配。

Codex Task

@TypeThe0ry TypeThe0ry marked this pull request as ready for review June 27, 2026 10:58
Copilot AI review requested due to automatic review settings June 27, 2026 10:58
@TypeThe0ry TypeThe0ry merged commit e6ba07a into main Jun 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

此 PR 通过在掼蛋队伍计分区域新增 .guandan-teams 容器,修复移动端媒体查询中依赖该容器的网格/隐藏样式无法生效的问题,从而让小屏端布局按既有 CSS 规则一致工作。

Changes:

  • 在掼蛋记分板中新增 <div class="guandan-teams"> 包裹队伍行列表
  • 保持 v-for="team in guandanTeams" 仍绑定在 .guandan-team-row 上,避免改变渲染行为

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread static/index.html
<strong>{{team.name}}</strong>
<span>{{team.members}}</span>
<div class="guandan-teams">
<div class="guandan-team-row" v-for="team in guandanTeams" :class="{active: team.score > 0}">
@TypeThe0ry TypeThe0ry deleted the codex/fix-guandan-team-score-area-styling branch June 27, 2026 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants