Skip to content
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

TLS_Cipher_String_Cheat_Sheet: added markdown tables #15

Merged
merged 2 commits into from
Feb 18, 2019
Merged

TLS_Cipher_String_Cheat_Sheet: added markdown tables #15

merged 2 commits into from
Feb 18, 2019

Conversation

0xb4lint
Copy link
Contributor

No description provided.

@0xb4lint
Copy link
Contributor Author

Do you think is it possible to add <br> as an allowed_elements to markdownlint rule MD033 - Inline HTML?

@righettod
Copy link
Member

The need for usage of <br> in your context is legit so you can use this syntax to add <br> in your MD table:

<!-- markdownlint-disable MD033 -->
Your MD table here
<!-- markdownlint-enable MD033 -->

It allow you to use HTML tags in your MD table, but please limit usage to <br> tag.

@righettod righettod added this to the Roadmap 2019 milestone Feb 18, 2019
@righettod righettod self-requested a review February 18, 2019 03:49
@righettod
Copy link
Member

You can also remove link to the image versions of the both tables if your contribution replace them.

@0xb4lint
Copy link
Contributor Author

I've commited the markdownlint-disable comments.
There is background colorization in the image versions, which we cannot reproduce in markdown.
Do you think is it OK to replace them?

@righettod
Copy link
Member

righettod commented Feb 18, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants