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

some more blacklisted attributes #323

Closed
wants to merge 1 commit into from
Closed

Conversation

LDAV
Copy link

@LDAV LDAV commented Mar 8, 2016

noshade is blacklisted for hr tags
align is blacklisted in p tags

in alternative, you can add an apply_filters:

private function get_blacklisted_attributes() {
  $args = array('style');
  return apply_filters('amp_blacklisted_attributes', $args);
}

noshade is blacklisted for hr tags
align is blacklisted in p tags
@mjangda
Copy link
Contributor

mjangda commented Jul 14, 2017

Closing as we've moved to a whitelist-based approach #600

@mjangda mjangda closed this Jul 14, 2017
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