microsoft / AdaptiveCards

A new way for developers to exchange card content in a common and consistent way.
https://adaptivecards.io
MIT License
1.75k stars 550 forks source link

[Mobile][Markdown support] [Markdown with the * does not work on mobile renderers] #4117

Closed jpjpjp closed 4 years ago

jpjpjp commented 4 years ago

Platform

Author or host

Host on Webex Teams.

Create a free Webex Teams account https://www.webex.com/team-collaboration.html to replicate using the Webex Teams iOS client. Create a space with cardSchool@webex.bot, and use the "Pick Another Lesson" button to select "Lesson 9 Buttons and Actions" to see how a Text element with markdown renders

Version of SDK

We have our own fork of AdaptiveCards@v1.2.6

Details

On Android a card that uses the '*' to create a list using markdon, does not render with bullets, but keeps the star:

AndroidMarkdown

On iOS it is even worse, as the newlines at the end of each list item seem to be ignored:

iOSMarkdown

The JSON for this card is here: https://developer.webex.com/buttons-and-cards-designer/lesson-9

shalinijoshi19 commented 4 years ago

This is likely a dup of #1984. @jwoo-msft / @paulcam206 / @almedina-ms can you guys confirm ?

ghost commented 4 years ago

Hi @jpjpjp. We have acknowledged this issue report. Please continue to follow this issue for updates/progress/questions.

shalinijoshi19 commented 4 years ago

The big ask is here is simply to support asterisk(*) for lists in our markdown story; While we are at it, we might want to support '+' also (Reference). We do say in our docs here that we do support bulleted lists though don't go into whether it's via hyphens only or also via asterisks '*' or '+' for that matter so it could be misleading there. @jwoo-msft / @paulcam206 what does it take to just simply support the 2 additional syntax notations for unordered lists in AC ?

jpjpjp commented 4 years ago

@jwoo-msft thanks for what looks like good progress on this? I wanted to bring one thing to your attention. Early on in our discussion on this, one suggestion was to use the hyphen instead of the star for list markdown.

We've found that that while the mobile renderers DO support the hyphen, the JS renderer does not, so adding the star to the mobile render will give a way to create platform agnostic cards.

matthidinger commented 4 years ago

Reopening this, @shalinijoshi19 asked about + support and indeed it seems like commonmark does support this as a format. It's not immediately obvious from their default page, https://commonmark.org/help/ but looking at the commonmark spec it does indeed show the + syntax:

https://spec.commonmark.org/0.29/#lists

If this is an easy enough addition to the open PR I'd agree let's just get it in?

jwoo-msft commented 4 years ago

The big ask is here is simply to support asterisk() for lists in our markdown story; While we are at it, we might want to support '+' also (Reference). We do say in our docs here that we do support bulleted lists though don't go into whether it's via hyphens only or also via asterisks '' or '+' for that matter so it could be misleading there. @jwoo-msft / @paulcam206 what does it take to just simply support the 2 additional syntax notations for unordered lists in AC ?

didn't see the note :-)

jwoo-msft commented 4 years ago

Reopening this, @shalinijoshi19 asked about + support and indeed it seems like commonmark does support this as a format. It's not immediately obvious from their default page, https://commonmark.org/help/ but looking at the commonmark spec it does indeed show the + syntax:

https://spec.commonmark.org/0.29/#lists

If this is an easy enough addition to the open PR I'd agree let's just get it in?

done

ghost commented 4 years ago

Hi @jpjpjp; We triaged and approved your bug for fixing. Please stay tuned for updates around fix progress and/or release vehicle.

ghost commented 4 years ago

Hi @jpjpjp. We have acknowledged this issue report. Please continue to follow this issue for updates/progress/questions.

ghost commented 4 years ago

:tada:AdaptiveCards@v1.2.10 has been released which fixes this issue.:tada:

Handy links: