Some attributes explicitly allowed are being removed about rehype-sanitize

oh placeholder is the only that works. the docs are correct nvm.

[leafac]

@wooorm: First, thanks for all your work on unified and related projects. It’s a joy to work with 😀

Second, I just got bitten by this 🤷

Since two years ago, when you marked this as wontfix, could you perhaps have reconsidered the situation? Perhaps make attribute case-insensitive (I believe that HTML attributes are case-insensitive, right?)?

Otherwise, would you accept a pull request noting that “playsInline, for example, should be playsInline” more prominently in the documentation? Perhaps both here and in https://github.com/syntax-tree/hast-util-sanitize#schema?

Thanks in advance.

[wooorm]

Hi @personaaleatoria, welcome! This behavior is expected: we support hast property names (autoFocus, srcSet, inputMode, spellCheck)

Thanks it worked. One note: I've tried placeHolder and it worked but in the documentation it says that it stays the same: https://github.com/syntax-tree/hast#propertyname.

when it says, "previous rules apply" does it mean that both placeholder and placeHolder are expected to work?

[wooorm]

Can you open a new issue, to improve the docs? (Or maybe even a PR!)

[robertu7]

Thanks, it works!

import rehypeSanitize, { defaultSchema } from 'rehype-sanitize'


...
  .use(rehypeSanitize, {
    tagNames: [
      ...defaultSchema.tagNames!,
      'iframe',
      'footer',
      'header',
      'audio',
      'source',
    ],
    attributes: {
      ...defaultSchema.attributes,
      a: ['href', 'ref', 'target'],
      img: ['src', 'srcSet', 'data*'],
 ...