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

add "cite" attribute to whitelist for <blockquote> elements #6094

Merged
merged 1 commit into from
Feb 24, 2016

Conversation

AprilArcus
Copy link
Contributor

Questions:

  • Would you like tests?
  • Would it be better if this whitelist were scoped to the node type of the parent element? cite is meaningless for any element but <blockquote>, but will be rendered into the DOM for any component per the general whitelist.

@AprilArcus AprilArcus changed the title add "cite" attribute to whitelist for <blockquote> elements (fixes #6084) add "cite" attribute to whitelist for <blockquote> elements Feb 23, 2016
@jimfb
Copy link
Contributor

jimfb commented Feb 24, 2016

Looks fine: https://jsfiddle.net/47zLu3jc/

jimfb added a commit that referenced this pull request Feb 24, 2016
add "cite" attribute to whitelist for <blockquote> elements
@jimfb jimfb merged commit 308badb into facebook:master Feb 24, 2016
@jimfb
Copy link
Contributor

jimfb commented Feb 24, 2016

Thanks @AprilArcus!

@zpao
Copy link
Member

zpao commented Feb 25, 2016

Thanks @AprilArcus! To answer your questions:

  1. We test these manually. Our automated tests use jsdom which is not 100% consistent with browsers.
  2. We've thought about making it scoped per node type but our ultimate goal has been to get rid of this whitelist entirely so we haven't invested in the idea (though there's a PR proposal doing that somewhere)

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

Successfully merging this pull request may close these issues.

4 participants