Repository navigation
Conversation
🦋 Changeset detectedLatest commit: f46cbc7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
dreyfus92
left a comment
There was a problem hiding this comment.
thanks for this! @btea 😄
one blocker, the fix lives in @clack/core but the changeset only bumps @clack/prompts. since prompts pins an exact core version, core needs a patch bump too or the fix never ships. could we add "@clack/core": patch?
small suggestion, could we move the disabled check into the find predicate? right now with a disabled delete and an enabled deploy, pressing d selects nothing since find stops at the disabled one first. with !disabled inside the predicate it'd pick deploy.
separately, the renderer doesn't style disabled options yet, so they look selectable and the initial cursor can land on one. fine as a follow up imo.
cc @43081j
|
Thank you for your review and suggestions. Adjustments have been made. |
What does this PR do?
Ignore key presses that match disabled options so they cannot be submitted through selectKey.
Type of change
Checklist
pnpm testpasses (or targeted tests for my change)pnpm formathas been runAI-generated code disclosure