Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Section tag is so commonly used. It will be very convenient to add them to the default block render map. #2074

Closed
gy-l opened this issue May 10, 2019 · 2 comments

Comments

@gy-l
Copy link

gy-l commented May 10, 2019

<section> tag is so commonly used. It will be very convenient to add them to the default block render map.expect so!

@claudiopro
Copy link
Contributor

Thanks for pointing this out @lisun0459, semantic tags like <section> and <article> are way more common nowadays than when the project was started. I'd be curious to see what is the distribution of tags supported by custom block render maps out in the wild.

This looks like a very good first issue for a community member who would like to contribute a code change to Draft.js! I'm more than happy to assist being the point of contact and doing a code review.

mmissey pushed a commit to mmissey/draft-js that referenced this issue Mar 24, 2020
Summary:
*Before* submitting a pull request, please make sure the following is done...

1. Fork the repo and create your branch from `master`.
2. If you've added code that should be tested, add tests!
3. If you've changed APIs, update the documentation.
4. Ensure that:
  * The test suite passes (`npm test`)
  * Your code lints (`npm run lint`) and passes Flow (`npm run flow`)
  * You have followed the [testing guidelines](https://github.com/facebook/draft-js/wiki/Testing-for-Pull-Requests)
5. If you haven't already, complete the [CLA](https://code.facebook.com/cla).

Please use the simple form below as a guideline for describing your pull request.

Thanks for contributing to Draft.js!

-

**Summary**

Fixes facebookarchive#2074
Updated the default block render map to have the section and article tags.
Pull Request resolved: facebookarchive#2212

Reviewed By: claudiopro

Differential Revision: D18246366

Pulled By: mrkev

fbshipit-source-id: 144f7721d92d6efcbb5c969ba992841eb9bd5f15
vilemj-Viclick pushed a commit to kontent-ai/draft-js that referenced this issue Jul 16, 2020
Summary:
*Before* submitting a pull request, please make sure the following is done...

1. Fork the repo and create your branch from `master`.
2. If you've added code that should be tested, add tests!
3. If you've changed APIs, update the documentation.
4. Ensure that:
  * The test suite passes (`npm test`)
  * Your code lints (`npm run lint`) and passes Flow (`npm run flow`)
  * You have followed the [testing guidelines](https://github.com/facebook/draft-js/wiki/Testing-for-Pull-Requests)
5. If you haven't already, complete the [CLA](https://code.facebook.com/cla).

Please use the simple form below as a guideline for describing your pull request.

Thanks for contributing to Draft.js!

-

**Summary**

Fixes facebookarchive#2074
Updated the default block render map to have the section and article tags.
Pull Request resolved: facebookarchive#2212

Reviewed By: claudiopro

Differential Revision: D18246366

Pulled By: mrkev

fbshipit-source-id: 144f7721d92d6efcbb5c969ba992841eb9bd5f15
@ab-pm
Copy link

ab-pm commented Nov 21, 2022

These appear to be missing from the documentation at https://draftjs.org/docs/advanced-topics-custom-block-render-map?

alicayan008 pushed a commit to alicayan008/draft-js that referenced this issue Jul 4, 2023
Summary:
*Before* submitting a pull request, please make sure the following is done...

1. Fork the repo and create your branch from `master`.
2. If you've added code that should be tested, add tests!
3. If you've changed APIs, update the documentation.
4. Ensure that:
  * The test suite passes (`npm test`)
  * Your code lints (`npm run lint`) and passes Flow (`npm run flow`)
  * You have followed the [testing guidelines](https://github.com/facebook/draft-js/wiki/Testing-for-Pull-Requests)
5. If you haven't already, complete the [CLA](https://code.facebook.com/cla).

Please use the simple form below as a guideline for describing your pull request.

Thanks for contributing to Draft.js!

-

**Summary**

Fixes facebookarchive/draft-js#2074
Updated the default block render map to have the section and article tags.
Pull Request resolved: facebookarchive/draft-js#2212

Reviewed By: claudiopro

Differential Revision: D18246366

Pulled By: mrkev

fbshipit-source-id: 144f7721d92d6efcbb5c969ba992841eb9bd5f15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants