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
Fix markdown-formatter customization accessibility #3101
Conversation
Generated by 🚫 Danger |
e265322
to
9788d74
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall I think we should not opt for this approach, it is better we expose our own API and do not be dependent on SwiftyMarkdown 👍
Sources/StreamChatUI/StreamSwiftyMarkdown/SwiftyMarkdown/SwiftyLineProcessor.swift
Outdated
Show resolved
Hide resolved
Sources/StreamChatUI/StreamSwiftyMarkdown/SwiftyMarkdown/SwiftyMarkdown.swift
Outdated
Show resolved
Hide resolved
Sources/StreamChatUI/StreamSwiftyMarkdown/SwiftyMarkdown/SwiftyMarkdown.swift
Outdated
Show resolved
Hide resolved
Sources/StreamChatUI/StreamSwiftyMarkdown/SwiftyMarkdown/SwiftyMarkdown.swift
Outdated
Show resolved
Hide resolved
Sources/StreamChatUI/Appearance+Formatters/MarkdownFormatter.swift
Outdated
Show resolved
Hide resolved
Btw @testableapple this is not bug, but actually an addition 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with most of Nuno's points - would be good to have our own small public API. We might get rid of this dependency eventually. Maybe we can even customize the font via our Fonts type and do the transformation internally?
a31a4db
to
15a98e6
Compare
Sources/StreamChatUI/Appearance+Formatters/MarkdownFormatter.swift
Outdated
Show resolved
Hide resolved
StreamChat XCMetrics
|
59f3dfb
to
bdc3051
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! ✅
Sources/StreamChatUI/Appearance+Formatters/MarkdownFormatter.swift
Outdated
Show resolved
Hide resolved
5efbcb4
to
0f3e026
Compare
Sources/StreamChatUI/Appearance+Formatters/MarkdownFormatter.swift
Outdated
Show resolved
Hide resolved
Sources/StreamChatUI/Appearance+Formatters/MarkdownFormatter.swift
Outdated
Show resolved
Hide resolved
Sources/StreamChatUI/Appearance+Formatters/MarkdownFormatter.swift
Outdated
Show resolved
Hide resolved
Sources/StreamChatUI/Appearance+Formatters/MarkdownFormatter.swift
Outdated
Show resolved
Hide resolved
73bb089
to
7be2141
Compare
Sources/StreamChatUI/Appearance+Formatters/MarkdownFormatter.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! ✅
Quality Gate passedIssues Measures |
🔗 Issue Links
Resolves #2908
🎯 Goal
Fix markdown-formatter customization accessibility
🛠 Implementation
Make the required stuff in
SwiftyMarkdown.swift
publically accessible🧪 Manual Testing Notes
# Test
)☑️ Contributor Checklist
🎁 Meme