Skip to content
This repository has been archived by the owner on Nov 30, 2023. It is now read-only.

Bugfix: Allow simultaneously adding cards and browsing cards #179

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

psii
Copy link

@psii psii commented Mar 8, 2021

Fixes a bug where simultaneous adding new cards and having the card browser open for editing stops the auto-fill feature from working correctly.

edit.EditManager wrongly assumes that there is only one Editor instance. The window for adding cards and the browsing window each instantiate their own editor.

Other small changes include replacing some legacy GUI hooks, see

psii added 3 commits March 8, 2021 12:32
Fixes the bug where simultaneous adding new cards and having the card
browser open for editing stops the auto-fill feature from working
correctly.

`edit.EditManager` wrongly assumes that there is only one `Editor`
instance.  The window for adding cards and the browsing window each
instantiate their own editor.
@jdlorimer jdlorimer self-assigned this Mar 29, 2021
@jdlorimer jdlorimer added this to the v1.0 milestone Mar 29, 2021
@jdlorimer jdlorimer added the bug label Mar 29, 2021
@jdlorimer
Copy link
Owner

Much appreciated.

I'm very likely to merge this once I've resolved the more critical issues and had some time to test the changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants