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.

Sign up for GitHub

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

Jump to bottom

fix(b-tooltip): Updated tooltip to work under shadowDOM #6997

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

fix(b-tooltip): Updated tooltip to work under shadowDOM #6997

wants to merge 2 commits into from

Commits on Jun 22, 2022

  1. fix(b-tooltip): Updated tooltip to work under shadowDOM 

    Added to dom utility methods
- isConnectedToDOM() checks if a target element is in the DOM and will check both Shadow and Regular DOM
- getShadowRootOrRoot() will return the target's root either the Shadow Root or DOCUMENT.body

Updated isVisibile() dom util to use new isConnectedToDOM() function

Updated the dom.spec.js unit tests for the two new dom utilities
Fixed the dom.spec.js to get the select() and selectAll() tests working
    @rasicoc
    rasicoc committed Jun 22, 2022
    Configuration menu
    Copy the full SHA
    752661a View commit details
    Browse the repository at this point in the history

  2. fix(b-tooltip): Updated tooltip to work under shadowDOM 

    Updated tests to get to 100% coverage of changed code + added coverage for isVisible()
Tightened the check in the getShadowRootOrRoot() method for document root
    @rasicoc
    rasicoc committed Jun 22, 2022
    Configuration menu
    Copy the full SHA
    5c169cc View commit details
    Browse the repository at this point in the history