Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cmdk UI improvements #176

Merged
merged 11 commits into from
Nov 11, 2024
Merged

Conversation

mrudulpatil18
Copy link
Contributor

@mrudulpatil18 mrudulpatil18 commented Nov 8, 2024

Issue: #153

Completed Changes:

  • Up Arrow/Down Arrow should allow you to select entries, "Enter" should cause that entry to be selected
  • "New Circuit Module" and "Create New Circuit Module" should be aliases for "Blank Circuit Module" (likewise for other template types)

Additional Changes:

  • added a footer section to the cmdk dialog

cc: @seveibar

Copy link

vercel bot commented Nov 8, 2024

@mrudulpatil18 is attempting to deploy a commit to the tscircuit Team on Vercel.

A member of the Team first needs to authorize it.

src/components/CmdKMenu.tsx Outdated Show resolved Hide resolved
@imrishabh18
Copy link
Member

@mrudulpatil18 Remove the bun lockfile from the PR, as your changes doesn't include any dependency update. It's not needed

@mrudulpatil18 mrudulpatil18 force-pushed the cmdk-improvements branch 2 times, most recently from bd910ee to 38fc04a Compare November 11, 2024 10:30
@mrudulpatil18
Copy link
Contributor Author

i have messed up a lot via my local setup, i'll fix this in some time

@mrudulpatil18
Copy link
Contributor Author

@mrudulpatil18 Remove the bun lockfile from the PR, as your changes doesn't include any dependency update. It's not needed

should be done now.

i hope i didn't mess up anything else in the process..

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