pell icon indicating copy to clipboard operation
pell copied to clipboard

Safari: RangeError: Maximum call stack size exceeded.

Open litemotiv opened this issue 6 years ago • 7 comments

I'm not sure if it's valid to use br as a defaultParagraphSeparator, but when using that it generates an error on Safari on this line:

if (firstChild && firstChild.nodeType === 3) exec(formatBlock, '<' + defaultParagraphSeparator + '>');else if (content.innerHTML === '<br>')

It seems to work okay in other browsers.

litemotiv avatar Jan 27 '19 14:01 litemotiv

Thanks for the report

jaredreich avatar Jan 28 '19 15:01 jaredreich

@litemotiv are you still experiencing this issue?

joeyhachemPB avatar Oct 18 '19 17:10 joeyhachemPB

@litemotiv are you still experiencing this issue?

I don't own a Mac personally so i cannot test this easily. I believe there have not been any updates to this code in Pell, so unless the latest version(s) of Safari have changed the behavior in this scenario the issue could likely still be present. At this point i cannot say anything definitive about it though...

litemotiv avatar Oct 19 '19 09:10 litemotiv

@litemotiv are you still experiencing this issue?

I don't own a Mac personally so i cannot test this easily. I believe there have not been any updates to this code in Pell, so unless the latest version(s) of Safari have changed the behavior in this scenario the issue could likely still be present. At this point i cannot say anything definitive about it though...

I see, thanks for replying

joeyhachemPB avatar Oct 21 '19 15:10 joeyhachemPB

Mac user here, problem still exists using safari

sidler avatar Oct 30 '19 13:10 sidler

This problem still exists in the current Safari 14.0. I could solve it locally by adjusting my configuration to read

pell.init({
    …
    defaultParagraphSeparator: 'br/',
    …
})

MikeMarschall avatar Nov 11 '20 12:11 MikeMarschall

Also experiencing this. Will try the fix mentioned above.

antony avatar Feb 10 '21 14:02 antony