Skip to content

Conversation

@SBoudrias
Copy link
Owner

Early PR adding a new package structure.

Still TODO:

  • Go through all packages and cover all string needing localization
  • Confirm this is good UX (like confirm prompt, not sure for all languages)

Comment on lines +5 to +10
confirm: {
yesLabel: '是',
noLabel: '否',
hintYes: '是/否',
hintNo: '是/否',
},
Copy link
Owner Author

Choose a reason for hiding this comment

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

@LitoMore Wondering if that's a good internationalization for non-latin languages 🤔

Copy link
Collaborator

@LitoMore LitoMore Nov 21, 2025

Choose a reason for hiding this comment

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

Maybe we can use bold text for the default choice.

hintYes: /否
hintNo: 是/

Or use with English:

yesLabel: '是(Yes)',
noLabel: '否(No)',
hintYes: '是(Y)/否(n)',
hintNo: '是(y)/否(N)',

@@ -0,0 +1,114 @@
<img width="75px" height="75px" align="right" alt="Inquirer Logo" src="https://raw.githubusercontent.com/SBoudrias/Inquirer.js/main/assets/inquirer_readme.svg?sanitize=true" title="Inquirer.js"/>
Copy link
Owner Author

Choose a reason for hiding this comment

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

Note to self: gotta edit README and adding a contribution guide.

@SBoudrias SBoudrias force-pushed the next-major branch 3 times, most recently from c5db0e9 to 8ca20d4 Compare November 16, 2025 22:12
Base automatically changed from next-major to main November 16, 2025 22:21
@@ -0,0 +1,152 @@
import {
Copy link
Owner Author

Choose a reason for hiding this comment

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

TBD might review having the index resolve the user locale based on process.env.LANG and having a @inquirer/prompts-i18n/create export 🤷🏻

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