Skip to content

Conversation

@Kuuuube
Copy link
Member

@Kuuuube Kuuuube commented Jan 4, 2026

Stops the user from accidentally double clicking the button or accidentally pressing the add hotkey twice. Button re-enable obv doesn't apply unless dupes/overwrite is allowed.

2.5s seems sane here to me but open to suggestions if this seems too much. If the user scans a word, adds it, scans another. Since the button is re-rendered, the second popup doesn't have any extra delay on adding.

Considered disabling then re-enabling after the card is created. But in some cases this could happen so fast that a user could still accidentally double click and end up adding two cards.

@Kuuuube Kuuuube requested a review from a team as a code owner January 4, 2026 17:21
@Kuuuube Kuuuube added kind/enhancement The issue or PR is a new feature or request area/ui-ux The issue or PR is related to UI/UX/Design area/anki The issue or PR is related to Anki integration labels Jan 4, 2026
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5659859f0e

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/anki The issue or PR is related to Anki integration area/ui-ux The issue or PR is related to UI/UX/Design kind/enhancement The issue or PR is a new feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant