Skip to content
This repository was archived by the owner on Aug 29, 2024. It is now read-only.

Fix the table formatting in mailing lists page #460

Closed
wants to merge 2 commits into from

Conversation

t8m
Copy link
Member

@t8m t8m commented Feb 1, 2024

The table formating was broken since we do not allow Markdown formatting parsing inside HTML anymore.

@levitte
Copy link
Member

levitte commented Feb 1, 2024

Unfortunately, our CSS style flushes tables tightly to the previous paragraph unless there's some other markup between them. Instead of removing the wrapping <p> and </p> completely, try to see if having <p>&nbsp;</p> between the table and the preceding paragraph, surrounded with empty lines.

@t8m
Copy link
Member Author

t8m commented Feb 1, 2024

@levitte I thought that but I do not see that here:

https://www.openssl.org/news/fips-cve.html
https://raw.githubusercontent.com/openssl/web/master/news/fips-cve.md - that does not contain any breaks/paragraphs before the table but the formatting looks ok on the web.

@t8m
Copy link
Member Author

t8m commented Feb 1, 2024

Hmm... could that problem be with the paragraph after the table? https://www.openssl.org/policies/general-supplemental/platforms.html

@t8m
Copy link
Member Author

t8m commented Feb 1, 2024

@levitte fixup pushed

@t8m t8m added the ready label Feb 2, 2024
@t8m
Copy link
Member Author

t8m commented Feb 2, 2024

Merged, thank you for the reviews.

@t8m t8m closed this Feb 2, 2024
openssl-machine pushed a commit that referenced this pull request Feb 2, 2024
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #460)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants