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

Phumudzo/fix code editor #226

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

nthangeniphumudzo
Copy link
Contributor

@nthangeniphumudzo nthangeniphumudzo commented Apr 11, 2023

Hi Ivan, this fix is related to this card. I have figured that the doublication of characters only happen when the "Autocompletion" properties are true on the 'CodeEditor' component so I have assigned them to false. Please approve my request if this doesn't cause any other discrepancies in the long run. I have also pushed minor style change for the imageAnnotation component.

@IvanIlyichev
Copy link
Contributor

Hi @nthangeniphumudzo. This bug requires a proper fix. You only disabled the autocompletion and did just in few places. We should fix the autocompletion instead of disabling

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.

None yet

2 participants