Skip to content

🐛 fix(banner): prevent title overflow on mobile with responsive f…#302

Merged
CuteLeaf merged 1 commit intoCuteLeaf:masterfrom
LovesAsuna:banner-title
Apr 18, 2026
Merged

🐛 fix(banner): prevent title overflow on mobile with responsive f…#302
CuteLeaf merged 1 commit intoCuteLeaf:masterfrom
LovesAsuna:banner-title

Conversation

@LovesAsuna
Copy link
Copy Markdown
Contributor

…ont-size

Type of change

  • Bug fix (a non-breaking change that fixes an issue)

Checklist

  • I have read the CONTRIBUTING document.
  • I have checked to ensure that this Pull Request is not for personal changes.
  • I have performed a self-review of my own code.
  • My changes generate no new warnings.

Related Issue

问题截图:
9d5e45918f51ff10ba1526e5aa0bd8c3_720

Changes

  • 1.1 在 src/styles/banner-title.css 中为 .banner-title 添加 overflow-wrap: break-wordword-break: break-word 属性
  • 1.2 在 src/styles/banner-title.css 中为 .banner-title 添加字体大小响应式计算:font-size: min(var(--banner-title-size, 3rem), 10vw) 并保留内联 font-size 作为 fallback
  • 1.3 在 src/styles/layout-styles.css 中更新各响应式断点的 .banner-title 样式,确保不再硬编码覆盖字体大小,而是使用 min(var(--banner-title-size), <breakpoint-max>) 方式

How To Test

Screenshots (if applicable)

修复后截图:
ce94fad52325126f46a9fe7cd5c127db_720

Additional Notes

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 18, 2026

Deploy Preview for demo-firefly ready!

Name Link
🔨 Latest commit 760c00a
🔍 Latest deploy log https://app.netlify.com/projects/demo-firefly/deploys/69e2fbdeb6e7410008a71e3d
😎 Deploy Preview https://deploy-preview-302--demo-firefly.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@CuteLeaf CuteLeaf merged commit 5a75705 into CuteLeaf:master Apr 18, 2026
9 checks passed
@LovesAsuna LovesAsuna deleted the banner-title branch April 18, 2026 05:09
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