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

fix(tooltip): fix isOpen and undefined containerClass [fixes #2257] #2262

Merged
merged 1 commit into from
Jul 26, 2017

Conversation

IlyaSurmay
Copy link
Contributor

@IlyaSurmay IlyaSurmay commented Jul 24, 2017

No description provided.

@IlyaSurmay IlyaSurmay requested a review from valorkin July 24, 2017 12:47
@codecov
Copy link

codecov bot commented Jul 24, 2017

Codecov Report

Merging #2262 into development will decrease coverage by 0.06%.
The diff coverage is 33.33%.

Impacted file tree graph

@@              Coverage Diff               @@
##           development   #2262      +/-   ##
==============================================
- Coverage        87.46%   87.4%   -0.07%     
==============================================
  Files               95      95              
  Lines             2681    2683       +2     
  Branches           358     358              
==============================================
  Hits              2345    2345              
- Misses             214     216       +2     
  Partials           122     122
Impacted Files Coverage Δ
src/tooltip/tooltip.directive.ts 67.7% <33.33%> (-1.45%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 64793c2...6b421e1. Read the comment docs.

@IlyaSurmay IlyaSurmay force-pushed the fix-tooltip-issues branch from b6bdef8 to 6b421e1 Compare July 25, 2017 08:19
@valorkin valorkin added this to the v1.8.1 milestone Jul 26, 2017
@valorkin valorkin merged commit 8664bb1 into development Jul 26, 2017
@valorkin valorkin deleted the fix-tooltip-issues branch July 26, 2017 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants