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

AST To NSAttributedString #81

Closed
wants to merge 64 commits into from
Closed

Conversation

johnxnguyen
Copy link
Owner

@johnxnguyen johnxnguyen commented Feb 28, 2018

FYI: this is superseded by #132

The PR introduces changes that allow an NSAttributedString to be generated directly from the AST, rather than through the intermediate step of generating HTML and then instantiating the attributed string. Other changes include the new DownStyle class to configure and style the attributed string.

@johnxnguyen johnxnguyen changed the title AST To NSAttribtuedString AST To NSAttributedString Feb 28, 2018
@iwasrobbed
Copy link
Collaborator

Hi @johnxnguyen 👋 This looks like quite the effort. I'd be curious to know why you took this approach (i.e. what does it offer over the alternative?). The more information you can provide, the better I can understand it if you re-open your PR.

Also, please note that all Down code (in Swift) must be licensed under MIT or it will not be acceptable to merge.

@johnxnguyen
Copy link
Owner Author

Hey @iwasrobbed ! My apologies, I mistakenly made the PR to your project, so you can ignore it. In any case, I'm a developer for Wire and have been tasked with integrating markdown into the app. We found generating the attributed text directly from HTML was a bit too limited as we needed a bit more control in how the attributed text is constructed. But I'd like to take this opportunity to thank you, this framework is fantastic!

@iwasrobbed
Copy link
Collaborator

Glad to hear it! As I mentioned to others, also check out BonMot if you haven't already

@iby
Copy link

iby commented Mar 9, 2019

@johnxnguyen Looks like a lot of effort indeed. Would you consider making a PR? It would make a great feature!

cc @lilyball @kengruven

@johnxnguyen
Copy link
Owner Author

Hi @ianbytchek, sure I would be happy to make a PR. Though, I would first like to go over it one more and clean up some things.

@iwasrobbed
Copy link
Collaborator

Thanks for the offer @johnxnguyen ! We'd happily welcome a PR and also happy to add you as a collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants