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
Add a new tag for line breaks (<br>) #687
Conversation
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.
(needs docs PR)
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 think a <newline>
alias would be nice as well.
...-minimessage/src/main/java/net/kyori/adventure/text/minimessage/tag/standard/NewLineTag.java
Outdated
Show resolved
Hide resolved
...-minimessage/src/main/java/net/kyori/adventure/text/minimessage/tag/standard/NewLineTag.java
Outdated
Show resolved
Hide resolved
...-minimessage/src/main/java/net/kyori/adventure/text/minimessage/tag/standard/NewLineTag.java
Outdated
Show resolved
Hide resolved
Not sure about a new alias like that. |
The majority of MiniMessage users will not be familiar with HTML. I do not see how adding a clearer tag would cause any more confusion. |
...inimessage/src/main/java/net/kyori/adventure/text/minimessage/tag/standard/StandardTags.java
Outdated
Show resolved
Hide resolved
I think |
2c7b212
to
9bb3a3e
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.
Final nitpicks! After this it's good to go :)
...-minimessage/src/main/java/net/kyori/adventure/text/minimessage/tag/standard/NewlineTag.java
Outdated
Show resolved
Hide resolved
...inimessage/src/main/java/net/kyori/adventure/text/minimessage/tag/standard/StandardTags.java
Outdated
Show resolved
Hide resolved
...inimessage/src/main/java/net/kyori/adventure/text/minimessage/tag/standard/StandardTags.java
Outdated
Show resolved
Hide resolved
...inimessage/src/main/java/net/kyori/adventure/text/minimessage/tag/standard/StandardTags.java
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.
Awesome, thank you for the PR and thanks for being so receptive and understanding of all my nitpicky comments! <3
Previous problem is here #686 :
Sometimes it's not possible or a bad idea to use
\n
for a line break. Maybe your message storage doesn't support that (or you'd have to escape it there).With this PR I want to add a
<br>
tag as alternative to \n. It'll replace<br>
with the Component for a new line.