Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Add tooltips to body to avoid css adjacency issues #218

Closed
wants to merge 1 commit into from

Conversation

ggoodman
Copy link

Currently, tooltip markup is added after the element on which the
directive is defined. As a result, any adjacency selectors based on
the element having a controlled set of siblings will be affected.

This change will add the tooltip element as a child of the body instead.

Currently, tooltip markup is added `after` the element on which the
directive is defined. As a result, any adjacency selectors based on
the element having a controlled set of siblings will be affected.

This change will add the tooltip element as a child of the body instead.
@joshdmiller
Copy link
Contributor

Thanks for this!

We've been discussing doing something similar in #139 but were waiting on a major refactor of the tooltip, which landed on Sunday but has yet to be merged (PR #210). We are also not sure which approach to take; do you want to hop on over to #139 and give your opinion?

Also, the build failed because we're currently migrating to Grunt 0.4, so don't worry about that.

@joshdmiller
Copy link
Contributor

I just opened a PR for this solution, but as an optional feature, in #254, so I'll go ahead and close this one. Thanks again.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants