Added Screenreader-Support with ariaId

[dsumer]

Added ariaId prop, which gets attached to the tooltips inner content container and can be used with aria-describedby for better accessibility and Screenreader-Support.

[coveralls]

Changes Unknown when pulling 741e1bd on dsumer:master into ** on react-component:master**.

[coveralls]

Changes Unknown when pulling 04b560a on dsumer:master into ** on react-component:master**.

[paranoidjk]

I see you just attached an id attribute to div，how does help for aria？

[dsumer]

For example, if you have an input field with an Tooltip around it you have currently no chance to link them for accessiblity purpose. With this id field you can do it like this:

                <Tooltip
                    ...
                    ariaId={this.props.name}
                    <input className={className}
                           type="text"
                           ...
                           aria-describedby={this.props.name}/>
                </Tooltip>

Now the Screenreader will associate the input field with the tooltip and read its contents when you focus the input.

[paranoidjk]

So describedby attribute just need a dom element id. Then for tooltip itself, i suggest you named this api to id

https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/ARIA_Techniques/Using_the_aria-describedby_attribute

[dsumer]

ok fine I will rename it to id .. then I will add an extra accesibility section to the README to describe how to work in harmony with screenreaders

[coveralls]

Changes Unknown when pulling b5e7030 on dsumer:master into ** on react-component:master**.

[coveralls]

Changes Unknown when pulling b5e7030 on dsumer:master into ** on react-component:master**.

[coveralls]

Changes Unknown when pulling b5e7030 on dsumer:master into ** on react-component:master**.

[coveralls]

Changes Unknown when pulling b5e7030 on dsumer:master into ** on react-component:master**.

[dsumer]

is this PR ready to be merged?

[benjycui]

Ref: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/ARIA_Techniques/Using_the_aria-describedby_attribute

👍

[dsumer]

me and my team are excited to see this in the next release of rc-tooltip :)
is there a chance that this PR will be merged?

[paranoidjk]

Sorry for the late, published + [email protected]