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

Highlight and write something on "Replace this string with your implementation." #596

Open
ghost opened this issue Mar 7, 2024 · 1 comment

Comments

@ghost
Copy link

ghost commented Mar 7, 2024

Related user(s):

No response

Related issue(s) or PR(s):

No response

Related project scope(s):

web-app UI

Bug description:

When I open an exercice and highlight the string "Replace this string with your implementation." :
1-highlight_my_answer

When I type a letter to replace the string, the letter appears after the two semicolons :
2-answer_appears_after_commas

To reproduce:

No response

Expected behavior:

When I type letter on the highlighted text, the letter should appears before the two semicolons :
expeted_behavior

learn-ocaml --version

v1.0.0

git describe --long --always --abbrev=40 --tags

No response

What OS do you use?

GNU/Linux

What OS version/distribution do you use?

Ubuntu 20.04.6 LTS

What browser(s) do you use with learn-ocaml?

Firefox, Edge

What browser(s) version did you used to reproduce the issue?

Firefox115.6.0esr; Edge version 121.0.2277.83

Screenshots (if need be):

No response

Additional context:

No response

@ghost ghost added the kind: bug label Mar 7, 2024
@ghost ghost changed the title Highlight and write something on "Replace this string with your answer." Highlight and write something on "Replace this string with your implementation." Mar 7, 2024
@AltGr
Copy link
Collaborator

AltGr commented Jul 29, 2024

Can indeed reproduce, thanks for reporting.
At the moment we are bound to a tweaked (and fixed) version of the Ace editor, which comes with many bugs and weirdnesses (in short, it is expected to allow our automatic-indentation mode, but provides no way to control concurrent operations, hence race conditions such as the one you are reporting).
The proper fix for this will probably be to plug-in a more modern and more reliable editor in place of Ace here, but that will require quite a bit of work.

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

No branches or pull requests

1 participant