-
Notifications
You must be signed in to change notification settings - Fork 751
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
Allow removal of rel for Terms and Privacy skin objects #5057
Conversation
…e skin Set the rel attribute of the hyperlink only if there is either no value set in the skin or a non blank value defined in the skin. If it is set to rel="" in the skin, the hyperlink will be rendered without a rel attribute.
Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>
Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>
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.
Looks good to me.
I moved the issue number from the title to the description so it auto-closes that issue.
I have added a summary to the Getter/Setter for the Rel attribute for the Privacy and Terms skin objects. ("Gets or sets the rel attribute of the Privacy/Terms skin object. Set it to an empty string to not render a rel attribute at all."). I made that change on my local branch ("Fix-for-issue-#5047"). Can I still submit that now? Or do I need to make a new branch and new PR for that? What would be the best approach? |
You can use the same branch, but it needs to be a new PR |
Set the rel attribute of the hyperlink only if there is either no value set in the skin or a non blank value defined in the skin. If it is set to rel="" in the skin, the hyperlink will be rendered without a rel attribute.
Fixes #5047
Summary