react/react-0.13.3.d.ts

to authors (@pspeter3 @vsiao AssureSign (account can't be detected) Microsoft (account can't be detected)). Could you review this PR?
👍 or 👎?

Checklist

• pass the Travis CI test?

👎

aria- attributes aren't set like this. They're also not whitelisted explicitly (see https://github.com/facebook/react/blob/master/src/renderers/dom/shared/HTMLDOMPropertyConfig.js#L26-L28).

@vsiao But doesn't typescript require them to be explicitly whitelisted? It seems like how react implements support isn't really relevant.

@vsiao, and @RyanCavanaugh final thoughts about this PR?

@wmill sorry if I wasn't clear. This isn't how aria attributes work. For example, instead of

React.createElement("div", { ariaHidden: true });

you must use

React.createElement("div", { "aria-hidden": true });

I don't know if TSX supports this. Maybe @RyanCavanaugh can chime in?

I think this was fixed separately...

#8027

#8236

#8236 would have over-fixed this and doesn't appear to still be present in HEAD.

I'm working on a different take on this pull that I'll submit separately.