Skip to content

Move bolding markdown outside of text link in help channel messages.#1144

Merged
ikuyarihS merged 2 commits into
python-discord:masterfrom
Numerlor:text-link-fix
Sep 5, 2020
Merged

Move bolding markdown outside of text link in help channel messages.#1144
ikuyarihS merged 2 commits into
python-discord:masterfrom
Numerlor:text-link-fix

Conversation

@Numerlor
Copy link
Copy Markdown
Contributor

@Numerlor Numerlor commented Sep 4, 2020

On some clients (android), the markdown is commonly not rendered properly within text links, in this case the bolding didn't happen and the markdown asterisks remainder as plaintext.
Moving them outside of the link text fixes the issue

On some devices the markdown gets rendered improperly,
leaving the asterisks in the message without bolding.
@Numerlor Numerlor requested a review from a team as a code owner September 4, 2020 18:28
@Numerlor Numerlor requested review from aeros and scragly and removed request for a team September 4, 2020 18:28
@ghost ghost added the needs 2 approvals label Sep 4, 2020
Copy link
Copy Markdown
Contributor

@MarkKoz MarkKoz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For what it's worth, both on Windows and Android, it didn't render as bold before your fix. However, unlike the screenshot, the asterisk weren't visible either.

@MarkKoz MarkKoz added a: frontend Related to output and formatting p: 3 - low Low Priority t: bug Something isn't working labels Sep 4, 2020
@Numerlor
Copy link
Copy Markdown
Contributor Author

Numerlor commented Sep 4, 2020

For what it's worth, both on Windows and Android, it didn't render as bold before your fix.

Hmm, it does render properly for me on windows; I guess it's inconsistent all over.

Copy link
Copy Markdown
Contributor

@ikuyarihS ikuyarihS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ghost ghost removed the needs 1 approval label Sep 5, 2020
@ikuyarihS ikuyarihS merged commit a3de2cd into python-discord:master Sep 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: frontend Related to output and formatting p: 3 - low Low Priority t: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants