IceCubesApp icon indicating copy to clipboard operation
IceCubesApp copied to clipboard

Bug: Incorrect rendering of post

Open pmonks opened this issue 2 years ago • 14 comments

This post is not being rendered correctly, as shown here:

image

Note: other clients I checked render it correctly.

pmonks avatar Jan 31 '23 23:01 pmonks

@xurble Do you have an idea? It's probably not your change but just the current lib failing at converting the HTML. I saw that from time to time.

Dimillian avatar Feb 01 '23 06:02 Dimillian

Looking at the thread, there's mention of zero width joining characters. I'll try and take a look this evening.

I'd actually like to take another crack at the markdown generation.

xurble avatar Feb 01 '23 17:02 xurble

This has been happening before any of @xurble changes. ~Is there an issue rendering due to EmojiText?~ Perhaps not? Looking further, I see it is using markdown.

EvilOne avatar Feb 01 '23 23:02 EvilOne

Here is one that I bookmarked a long time ago that has an issue displaying -> https://newsie.social/@jeff/109716493698039652

It renders like this:

EvilOne avatar Feb 02 '23 00:02 EvilOne

FWIW the zero width joiners were breaking HTML2Markdown

xurble avatar Feb 02 '23 11:02 xurble

Nice find!

EvilOne avatar Feb 02 '23 11:02 EvilOne

Are there different kinds of zero width joiners? This screenshot show that my post is rendered correctly, there is a zwsp between "#tag" and "s"

image

sothawo avatar Feb 02 '23 12:02 sothawo

Addition: I enter it as a unicode character, not as an html entity

sothawo avatar Feb 02 '23 12:02 sothawo

Can you post the url for the screenshotted post please?

xurble avatar Feb 02 '23 14:02 xurble

https://digitalcourage.social/@sothawo/109795642095913757 new one, the original was not (only visible for me)

sothawo avatar Feb 02 '23 14:02 sothawo

Thank you

xurble avatar Feb 02 '23 14:02 xurble

@EvilOne :

Well yours is fixed in the newer version, so that's good :)

Simulator Screen Shot - iPhone 14 Pro - 2023-02-02 at 14 38 38

xurble avatar Feb 02 '23 14:02 xurble

well, it didn't break the current version - and did not break previous ones, I use always use this when I have pluralized tags.

sothawo avatar Feb 02 '23 14:02 sothawo

@sothawo Your working zero width is (hex) 80 8B The breaking one is E2 80 8D

I don't know why HTML2Markdown chokes on one but not the other.

xurble avatar Feb 02 '23 15:02 xurble

This can be closed, it's fixed

xurble avatar Mar 01 '23 23:03 xurble