Skip to content

Conversation

@x-magic
Copy link
Contributor

@x-magic x-magic commented May 9, 2024

There has been a lot of upstream updates since my previous PR so I thought it's time to update!

Mainly focused on the interpretation quality and use of language. A few cosmetic changes (such as "File Transfer" in BLE screen as there's already a BT icon in front of it), too.

我先放draft PR吧,我得再过一遍UI看看还有没有要改的地方。屏幕太小了很多地方不太能挤得下。而且最近才有的二奶机,终于舍得为了一些字串而重置设置和格式化了😏

@netlify
Copy link

netlify bot commented May 9, 2024

Deploy Preview for aquamarine-heliotrope-f610d4 ready!

Name Link
🔨 Latest commit 431b266
🔍 Latest deploy log https://app.netlify.com/sites/aquamarine-heliotrope-f610d4/deploys/6646abe9aa559d0008c68844
😎 Deploy Preview https://deploy-preview-252--aquamarine-heliotrope-f610d4.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 site configuration.

@solosky
Copy link
Owner

solosky commented May 9, 2024

感谢贡献,友情提醒下:需要把i8n.csv的修改提到PR里面。

@x-magic x-magic marked this pull request as ready for review May 17, 2024 01:01
@x-magic
Copy link
Contributor Author

x-magic commented May 17, 2024

感谢贡献,友情提醒下:需要把i8n.csv的修改提到PR里面。

改好了,润色也在实机测试过了

Copy link
Owner

@solosky solosky left a comment

Choose a reason for hiding this comment

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

一些小的修改建议

[_L_APP_CHAMELEON_CARD_SET_NICK_SUCCESS] = "Nickname is Set",
[_L_APP_CHAMELEON_CARD_SET_NICK_FAILED] = "Set Nickname Failed",
[_L_APP_CHAMELEON_CARD_INPUT_NICK] = "Enter Nickname:",
[_L_APP_CHAMELEON_CARD_DATA_LOAD] = "Load Card Data...",
Copy link
Owner

Choose a reason for hiding this comment

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

建议保留 Load.. 和 Save.. 后面加card data 感觉有点多余,也不能强调是Load还是Save

[_L_SHOW_QRCODE] = "Display QR Code",
[_L_AMIIBO_KEY_UNLOADED] = "Amiibo Key Not Found",
[_L_UPLOAD_KEY_RETAIL_BIN] = "Upload key_retail.bin file\nto the root directory of\nthe external storage.",
[_L_KNOW] = "Understood",
Copy link
Owner

Choose a reason for hiding this comment

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

建议改成OK

[_L_APP_AMIIBOLINK] = "AmiiboLink",
[_L_APP_BLE] = "BLE File Transfer",
[_L_APP_BLE_TITLE] = "BLE File Transfer",
[_L_APP_BLE_TITLE] = " File Transfer",
Copy link
Owner

Choose a reason for hiding this comment

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

多了一个空格

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的我修改完实机编译一下看看

文件传输那里多个空格是因为简体中文有没有空格都看上去不突兀,但英文版的蓝牙图标连着后面的字串就显得很突兀。把上一版里的BLE去掉也是同样的原因,因为那个界面里这行是居中的,加上BLE的话就太长了。要不你考虑一下这一处源码里图标后面总是跟一个空格?

类似的还有个地方想不起来在哪儿了,一般来讲短省略号都是三个点,但那一处源码里hardcoded有两个点没有用printf,后来也就没改了。

@solosky solosky changed the base branch from main to develop August 21, 2024 02:57
@wkelebn
Copy link

wkelebn commented Aug 7, 2025

加入繁中amiidb_game.zip
我加了一些繁中,希望幫得上幫

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.

3 participants