19
submitted 3 months ago by baconeater@lemm.ee to c/lemmyconnect@lemmy.ca
you are viewing a single comment's thread
view the rest of the comments
[-] kuro_neko@lemmy.ca 16 points 3 months ago

Hi, glad you're loving Connect! I pushed an update yesterday (v187) that should address the markdown issues noted. Hopefully @gedaliyah@lemmy.world will update at some point :)

[-] Morphit@feddit.uk 6 points 3 months ago

10/10 now! ๐ŸŽ‰
Thanks for the updates.

[-] gedaliyah@lemmy.world 5 points 3 months ago

Wow, great work!

Personally, I don't love the spoiler text being hidden instead of collapsed, but that seems like more of a style choice than function issue. It shows and hides as it should.

I know for sure that there are communities who would appreciate spoilers in this style and I hope that they find your app!

[-] Morphit@feddit.uk 2 points 3 months ago

Yeah, there also seems to be a bit of an issue with formatting embedded inside spoilers. In your comment there are inline images in a spoiler block. They load on the web, but not in Connect. I wonder how other apps deal with that. Maybe another round of testing is in order soon!

I think it'd be nice to have the hidden block if the text is just one line, but if there are images or many lines then just make it expandable and load images when opened. I have no idea how much work that is to implement though.

[-] gedaliyah@lemmy.world 2 points 3 months ago

I figured out pretty quickly that there is no comprehensive way to test markdown on Lemmy because of differing standards, and because of an infinite combination of different tags, like images inside of spoilers, as you say. I tried to stick to the most common uses I see and which have been referenced and used widely for a long time (which is very subjective but I had to start somewhere). In theory, every combination should work, but we (Lemmy as a whole) don't even have the basics consistent.

[-] i_am_not_a_robot@feddit.uk 2 points 3 months ago

Yes, markdown inside spoilers is something which would be good to see fixed.

this post was submitted on 03 Aug 2024
19 points (95.2% liked)

Connect for Lemmy App

2661 readers
4 users here now

A community for the mobile app Connect for Lemmy.

Links

founded 1 year ago
MODERATORS