Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] Expand Markdown support #86

Open
mm2270 opened this issue May 23, 2022 · 2 comments
Open

[Feature Request] Expand Markdown support #86

mm2270 opened this issue May 23, 2022 · 2 comments
Labels
dependencies Pull requests that update a dependency file open to community An issue that will not be worked by IBM but is open for external developers

Comments

@mm2270
Copy link

mm2270 commented May 23, 2022

Is your feature request related to a problem? Please describe.
I've found that one or two common ways of using Markdown aren't supported in the popup text view.
For example, in Markdown, to make text both bold and italic, you add 3 asterisks before and after the text.

Example:

This text is bolded
This text is italicized
This text is both bolded and italicized

As you can see, this works on github, but in IBM Notifier, it seems to ignore the extra asterisk and just makes the text bold.

Describe the solution you'd like
I'd like to see some more Markdown be supported in future versions if this is possible to do. I believe another format for Markdown that isn't fully supported is numbered lists. I get very inconsistent results when trying to use it with IBM Notifier.

Describe alternatives you've considered
Using the html accessory view can be used as an alternative, but then if I want to use the html accessory view for something else, it complicates setting up the dialog.

Additional context
None

@SMartorelli
Copy link
Member

Hi @mm2270 to enable markdown support we use an open source library called SwiftyMarkdown. This library ease the translation from Markdown strings to Attributed Strings (the string format used inside macOS/iOS apps to format text). I see that this library already has a PR open about fixing this issue with the bolded and italicized text. In case they don't merge that PR we'll might think about forking it.

@SMartorelli SMartorelli added dependencies Pull requests that update a dependency file question Further information is requested labels May 24, 2022
@mm2270
Copy link
Author

mm2270 commented May 24, 2022

Ok thank you for the update!

I see the PR for that is 11 months old now. I wonder if someone is still maintaining that library. 🤔 Hopefully so.

@SMartorelli SMartorelli added open to community An issue that will not be worked by IBM but is open for external developers and removed question Further information is requested labels Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file open to community An issue that will not be worked by IBM but is open for external developers
Projects
None yet
Development

No branches or pull requests

2 participants