Skip to content

fix(campaign): logged-in discussion polish (spacing, alignment, centering)#5980

Merged
yingshinlee merged 5 commits into
developfrom
fix/discussion-logged-in-polish
Jun 18, 2026
Merged

fix(campaign): logged-in discussion polish (spacing, alignment, centering)#5980
yingshinlee merged 5 commits into
developfrom
fix/discussion-logged-in-polish

Conversation

@yingshinlee

Copy link
Copy Markdown
Collaborator

在 ICU 登入態測出的討論區排版問題修正(接續 #5978)。base = develop

修正項目

  1. 「查看全部 N 則留言」置中(原本偏左)
  2. 打字框與第一則留言的間距:inlineFooter 變體原本 margin-bottom:0,導致彈窗內表單緊貼第一則留言 → 補回 sp16
  3. 打字框高度:送出鈕移進框內後整體偏高 → 把框內編輯器 min-height 縮為 3rem(透過全域 class u-content-comment 限定,圍爐不受影響
  4. 二級評論內文對齊.contentContainer 縮排原本為 32px 頭像設計(sp40),二級用 24px 頭像 → 文字往後縮半格;新增 compact 縮排(sp32)對齊名稱,僅限 campaignDiscussion

待辦(需實機微調)

  • 一級/二級「暱稱↔ID」間距觀感不一致:研判與頭像大小差異(32 vs 24)/窄欄換行有關,待在 Vercel preview 登入態確認後再調

驗證

  • tsc --noEmit
  • ⚠️ 討論區僅在 production build render(本機 next dev 不 render)→ 請在本 PR 的 Vercel preview 登入態驗收(這次特別要測登入後+有二級留言的情境)

…ring)

Issues found once testing the logged-in discussion on ICU:
- centre the "view all N comments" link in the module
- restore a gap below the comment box (the inline-footer variant had
  margin-bottom:0, gluing the first comment to the box) and compact the
  editor min-height inside that box (scoped via the global u-content-comment
  class so 圍爐 is untouched)
- align 24px-avatar reply text under the author name (the .contentContainer
  indent was sized for the 32px avatar) — scoped to campaignDiscussion

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@yingshinlee yingshinlee requested a review from a team as a code owner June 18, 2026 01:30
@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
matters-web Ready Ready Preview Jun 18, 2026 7:35am

Request Review

…pping

- revert the centered "view all N comments" link back to left-aligned, matching
  the site's ViewMoreComment convention (text-align: start); nothing on the
  site centers this line
- pass nameNoWrap to the comment author for campaign discussion so the
  display name + @id stay on one line; in the indented reply column they
  otherwise wrap to two lines and the name↔id gap looks cramped

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- the counter used var(--color-red), which is undefined, so it never turned
  red past the limit; use --color-negative-red, the site-wide form-error red
  (ArticleCommentForm / Form.Field use the same)
- the campaign submit label was 15px; drop to 14px to match the site's comment
  submit buttons (ArticleCommentForm), scoped so circle forms keep their size

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ange)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@yingshinlee yingshinlee merged commit e45ac38 into develop Jun 18, 2026
4 checks passed
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