-
Notifications
You must be signed in to change notification settings - Fork 40
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 html color #15
Conversation
It was noticed it in the comment of the private tag() method, but not really possible to do. Now it is ;)
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.
Thanks for the PR. Please adjust the javadocs.
* version will get only the content. | ||
* | ||
* @param content the text content | ||
* @param content the text content |
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.
Duplicated @param content, should be color.
Sorry, I duplicated the param but forgot to change it... Done. |
I'm having a second though about this. Reviewing the Builder class I see there is no other method like the one you propose. It seems a bit ad-hoc to me. Maybe a better solution would be just to make the |
I'm publishing the Do you think that's enough? |
I completely agree with you. I didn't want to break the public/private scope of your methods but it's more efficient in this way. Thank you. |
Great. I'm closing this PR and publishing a new version. |
It was noticed it in the comment of the private tag() method, but not
really possible to do. Now it is ;)