Skip to content

feat(ickb-i18n): complete i18n support for iCKB core module and UI enhancements#5

Merged
Hanssen0 merged 6 commits intockb-devrel:mainfrom
CKBFansDAO:fix/ickb-i18n
Mar 18, 2026
Merged

feat(ickb-i18n): complete i18n support for iCKB core module and UI enhancements#5
Hanssen0 merged 6 commits intockb-devrel:mainfrom
CKBFansDAO:fix/ickb-i18n

Conversation

@yixyxiu
Copy link
Copy Markdown

@yixyxiu yixyxiu commented Mar 16, 2026

Overview

This PR completes the internationalization support for the iCKB core module and includes several UI improvements for better user experience.

Key Changes

🌐 i18n Core Module Support

  • Refactor txInfo messages: Convert all hardcoded English strings from iCKB SDK to structured i18n tokens
  • Translation keys: Add 18 new translation keys (11 info + 7 error) under ickbTxInfo namespace
  • UI updates: Update components to render structured tokens using t() function

🎨 UI Enhancements

  • Clickable history items: Convert transaction history items to anchor tags with blockchain explorer links
  • Input styling: Standardize input field styling across all forms with consistent theme colors, add hover:bg-white/10 for better interactivity
  • List display: Render transaction info as bulleted lists instead of joined text for better readability
  • Error highlighting: Display error messages in red within info lists

📷 Screentshot

  • Before:
image
  • After:
image

yixyxiu added 6 commits March 16, 2026 12:14
- Convert all hardcoded info/error strings to structured tokens
- Update UI components to render tokens via t()
- Change "Receive" → "Will receive" in deposit/withdraw forms for better UX clarity
- Split creatingLimitOrder into two keys to avoid mixing English text in Chinese
- Change "Will receive" → "You will receive" for better UX tone
- Change "将获得" → "您将获得" for more polite Chinese expression
- Render transaction info as bulleted list instead of joined text for better readability
- Highlight error messages in red within the info list
- Convert DashboardHistoryItem to anchor tag with blockchain explorer link
- Convert IckbHistoryOrderItems to anchor tag with blockchain explorer link
- Add txHash field to RecentOrder type for proper linking(ickb)
- Update queries to include txHash in order data(ickb)
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 16, 2026

Deploy Preview for nervdao ready!

Name Link
🔨 Latest commit 8c41927
🔍 Latest deploy log https://app.netlify.com/projects/nervdao/deploys/69b84b607976680008514d27
😎 Deploy Preview https://deploy-preview-5--nervdao.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.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 16, 2026

Deploy Preview for testnetvdao ready!

Name Link
🔨 Latest commit 8c41927
🔍 Latest deploy log https://app.netlify.com/projects/testnetvdao/deploys/69b84b60f594b90008d23baf
😎 Deploy Preview https://deploy-preview-5--testnetvdao.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.

Copy link
Copy Markdown
Member

@Hanssen0 Hanssen0 left a comment

Choose a reason for hiding this comment

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

LGTM

@Hanssen0 Hanssen0 merged commit 9488b7c into ckb-devrel:main Mar 18, 2026
8 checks passed
@yixyxiu yixyxiu deleted the fix/ickb-i18n branch April 8, 2026 04:14
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