Skip to content

Fix CJK content overflowing story page layout#687

Merged
realproject7 merged 1 commit intomainfrom
task/686-cjk-overflow
Mar 31, 2026
Merged

Fix CJK content overflowing story page layout#687
realproject7 merged 1 commit intomainfrom
task/686-cjk-overflow

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Adds overflow-wrap: break-word, word-break: break-word, and overflow-x: hidden to .ruled-paper in globals.css
  • Fixes Chinese/Japanese text overflowing the container and pushing sidebar off-screen

Fixes #686

Test plan

  • https://plotlink.xyz/story/40 (Chinese) — text wraps within container, sidebar visible
  • https://plotlink.xyz/story/39 (Japanese) — text wraps within container, sidebar visible
  • English stories unaffected — normal wrapping preserved
  • Korean stories unaffected
  • Reading Mode CJK content wraps correctly
  • Mobile view CJK content wraps correctly
  • npm run build passes ✅

🤖 Generated with Claude Code

Add overflow-wrap, word-break, and overflow-x: hidden to .ruled-paper
so Chinese/Japanese text wraps correctly without pushing sidebar off-screen.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 31, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Mar 31, 2026 7:43pm

Request Review

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

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

LGTM. Minimal 3-line CSS fix matching issue #686 exactly: overflow-wrap, word-break, and overflow-x: hidden on .ruled-paper. No unrelated changes.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

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

Verdict: APPROVE

Summary

This is a minimal, targeted CSS fix that matches issue #686. Adding overflow-wrap, word-break, and overflow-x: hidden on .ruled-paper addresses the reported CJK overflow without broadening scope.

Findings

  • None.

Decision

Approving because the change is narrowly scoped to the affected story container and aligns with the issue's requested fix and verification intent.

@realproject7 realproject7 merged commit 20f9256 into main Mar 31, 2026
5 checks passed
@realproject7 realproject7 deleted the task/686-cjk-overflow branch March 31, 2026 19:48
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.

Fix CJK (Chinese/Japanese) content overflowing story page layout

2 participants