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

README: Allow align attributes #3297

Merged
merged 1 commit into from
Feb 25, 2021
Merged

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Feb 14, 2021

Resolves #2151

As far as I'm aware there is no way that this could be used in any malicious way and align is already allowed on e.g. img elements by default in the ammonia crate.

@Turbo87 Turbo87 added C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works A-backend ⚙️ labels Feb 14, 2021
@locks
Copy link
Contributor

locks commented Feb 25, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Feb 25, 2021

📌 Commit 06660eb has been approved by locks

@bors
Copy link
Contributor

bors commented Feb 25, 2021

⌛ Testing commit 06660eb with merge bf63549...

@bors
Copy link
Contributor

bors commented Feb 25, 2021

☀️ Test successful - checks-actions
Approved by: locks
Pushing bf63549 to master...

@bors bors merged commit bf63549 into rust-lang:master Feb 25, 2021
@Turbo87 Turbo87 deleted the markdown-align branch February 25, 2021 17:04
bors added a commit that referenced this pull request Sep 17, 2021
Extract `cio_markdown` subpackage

This allows us to isolate our markdown rendering code in a dedicated package, which could potentially have a positive effect on the compile times of the full project. It also means we can iterate on this part of the code independent from the other parts (see #3861 or #3297), and with a well-defined API interface.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-backend ⚙️ C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works
Projects
None yet
Development

Successfully merging this pull request may close these issues.

README: Add support for basic HTML styling
5 participants