Card with lots of new lines hangs

bentsai · September 21, 2021, 4:13pm

I was trying to reproduce Long `Ctrl + Enter` breaks the UI and I ran into another (maybe related) bug. Steps:

(On Vivaldi browser) create a new card:

Hello







World

As soon as I hit backspace, the browser hangs. If I wait long enough (2-3 minutes), the JavaScript finishes running. I did a profile I saw that it was related to trim() being called a lot?

I went to my phone and edited this same card, and I was able to delete characters from the name there with no problems.

This occurs on both Vue 2 and 3

bentsai · September 22, 2021, 3:00am

I made a recording of it using this new tool called Replay: https://app.replay.io/recording/9af96933-5c56-45db-89be-82de3f03f5bc

bentsai · September 22, 2021, 3:42am

I reproduced in Vivaldi and Firefox. Safari on desktop and iOS both are okay.

pirijan · September 24, 2021, 3:18am

Sweet, I’ll get to this in my next bug fixing pass

bentsai · October 28, 2021, 2:17am

I just reproed this bug again and on Microsoft Edge, it completed locked up the browser. I couldn’t load the space anymore. I had to go delete the card in Safari and delete this space from localStorage.

pirijan · October 28, 2021, 3:11pm

Thanks for the reminder, will look into today

pirijan · October 28, 2021, 3:45pm

i’m having trouble repro-ing this,

using vivaldi
i copy in the quoted text above








World

i hit backspace (and tried other inputs too)
the browser remains responsive, instead of locking up (i tried this on both dev and prod envs)

did i miss something?

bentsai · October 28, 2021, 4:22pm

Can you try a few more empty lines? You are right, when I pasted that text, it was responsive. But after I added about 10 more lines, it started to slow down. If I add more, it starts to lock up.

pirijan · October 28, 2021, 4:36pm

cool was able to repro,
fix released