Skip to content

Real-device fixes: Apple emoji, mobile fit, SSR i18n, Node 24#11

Merged
ififi2017 merged 1 commit into
mainfrom
feat/share-countdown
Jul 6, 2026
Merged

Real-device fixes: Apple emoji, mobile fit, SSR i18n, Node 24#11
ififi2017 merged 1 commit into
mainfrom
feat/share-countdown

Conversation

@ififi2017

Copy link
Copy Markdown
Owner

iPhone 真机反馈修复:分享图片改用 Apple 风格 emoji、移动端弹窗一屏适配、跳过 SSR 期翻译请求(403)、升级 Node 24。

Follow-ups from real-device (iPhone) testing of the share feature.

- Emoji missing in the share image on iOS: canvas fillText doesn't rasterize
  system color emoji on iOS Safari. Bundle the 8 mood emojis as Apple-style
  PNGs (public/emoji/*.png, from iamcal/emoji-data img-apple-160) and drawImage
  them instead, with a text fallback — consistent Apple artwork on every
  device. Adds a codepoint to each mood; middleware now excludes /emoji/ (like
  /locales/) so the assets aren't caught by the locale redirect.
- Share dialog didn't fit one screen on mobile: shorter preview on small
  screens (240px, 300px on sm+) and tighter dialog padding/gap on mobile.
- Build log noise "Failed to load resources: translation 403": the client
  i18n module fetched translations over HTTP during SSG, hitting the
  production domain (403) needlessly. Guard the initial load to the browser;
  SEO/metadata already read translations from the filesystem.
- Node 20 is deprecated on Vercel (builds fail after 2026-10-01): set
  engines.node to 24.x, bump @types/node and CI to Node 24.

Note: Apple emoji artwork is © Apple Inc.; the images are redistributed via
iamcal/emoji-data (common practice for web, but not officially licensed).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 6, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
off-work-30 Ready Ready Preview, Comment Jul 6, 2026 1:41am

@ififi2017
ififi2017 merged commit 5f6cf33 into main Jul 6, 2026
3 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