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

#535 search parents of click target in body mode #613

Closed
wants to merge 4 commits into from
Closed

#535 search parents of click target in body mode #613

wants to merge 4 commits into from

Conversation

idc-developer
Copy link

This will fix #535, reissued pull request after build sytem change to rollup

Marco van Meegen added 3 commits June 18, 2020 13:04
BREAKING CHANGE: if a tooltip applies to a dom tree instead of a single element, a click on tree's
children will now result displaying a tooltip if a data-tip attribute is found in parent chain of
clicked dom element. This might affect code where children are not contained in the bounds of their
parents

#535
@knubie
Copy link

knubie commented Jul 8, 2020

@idc-developer Are you using this branch in production? Do you have the dist hosted somewhere like on github? I'd like to use your branch in production if possible.

@idc-developer
Copy link
Author

Yes we do. We have forked it and committed the dist into our fork master: https://github.com/idc-developer/react-tooltip
Marco

@aa333
Copy link

aa333 commented Jan 26, 2021

@idc-developer thank you for your work. Would it be possible to add the data-tip-disable short return to your walker loop? I'd like to be able to override/disable tooltips on some child elements, i.e. if you meet data-tip-disable, you don't need to go up.

Although it's debatable since someone might expect that when you disable tooltip on child, tooltip on parent should take its place.

@idc-developer
Copy link
Author

@idc-developer thank you for your work. Would it be possible to add the data-tip-disable short return to your walker loop? I'd like to be able to override/disable tooltips on some child elements, i.e. if you meet data-tip-disable, you don't need to go up.

Although it's debatable since someone might expect that when you disable tooltip on child, tooltip on parent should take its place.

The name seems not well fitting. Maybe you should name it like browser events "data-tip-stop-propagation". Currently I have no time to integrate this, but you could fork and do it yourself and create a pull request here.

This pull request was closed.
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.

in bodyMode search parents of current target for tooltips
4 participants