Use of space key causing weird cursor behaviour

[maxkratz]

Hi,

I've tested etherpad as docker container and I found a bug that shows up even if no special configuration gets applied.

The cursor jumps back to the beginning of a line if the user types in a space. I can reproduce this with docker-image 'latest', '1.8.0' and '1.7.5' on Chromium/Firefox (Ubuntu 18.04) and iPad Safari. Changing the theme to 'no-skin' did not help. (Furthermore there are no plugins installed at all.)

You will find a gif attached to this message for further description.

[muxator]

Reopening this issue thanks to a comment in #3663 by @sudoman.

This issue is probably related to #3663, but it may not be the same.

[muxator]

@maxkratz, is it possible for you to test #3729 and see if this problem is solved?

If you need help in integrating those changes, fell free to write here.

[maxkratz]

@muxator I've pulled your pr and built a new docker-image out of it (using the provided Dockerfile). As far as I can tell, the issue persists and nothing changed. (Tested with chromium and firefox, both on linux.)

[maxkratz]

@muxator I think I made a mistake. In my previous tests I did not clear the caches of all my browsers.
As soon as I always empty the cache(s), the Docker image with tag '1.8.0' (and older) does the job.

(Furthermore I am able to build a local Docker image out of this repository with tag '1.8.0' and everything works fine. If I use the latest version of the develop branch, the bug occurs!)

So in my opinion the error must be a change in code after version '1.8.0'.

[maxkratz]

@muxator I've pulled your pr and built a new docker-image out of it (using the provided Dockerfile). As far as I can tell, the issue persists and nothing changed. (Tested with chromium and firefox, both on linux.)

My comment above was wrong as I did not clear the browsers cache.

@muxator I've tested your pr again with proper clearing of my browsers cache. Your change does indeed resolve the problem!

I was able to build a local Docker image from your pr and this version just works as expected. Thank you for your help!

[muxator]

Thanks @maxkratz for validating it. As you can see these sort of issues are finky to debug, and more eyeballs are always a good protection net.

I'll wait for further tests by other users and merge #3729.

p.s.: I am starting to suspect that the Etherpad's caching behaviour needs a rework