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
Remove forced coloring of blog links #3140
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.
How did you create that link? I couldn't find a way to add a link using the admin without it being wrapped in a paragraph tag. |
@alanmoo guess we should update our fake data generator so it's more realistic 😂 |
See #2786 :) |
c320ff7
to
bee813a
Compare
Since links are always inside `p` elements, we don’t need to explicitly set the color.
bee813a
to
2033e53
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.
I just destroyed and recreated the review app. https://foundation-mofostaging-pr-3140.herokuapp.com/en/blog/role-suggest-defense/ There's no extra closing paragraph tag in there, just the one I added in the test fixture to replicate the wrapping Wagtail's rich text editor does on save. |
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.
The mismatched <p>
/</p
> validation error seems to be resolved on the review app now. R+
Since links are always inside
p
elements, we don’t need to explicitly set the color.Closes #3127