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(autoclose): fix popup components auto closing in Safari 13 #3454

Merged

Conversation

maxokorokov
Copy link
Member

This fixes multiple issues related to popup components auto closing.
Don't use touch events anymore as everything works fine with mouseup/down

Fixes #3446
Fixes #3437
Fixes #3412
Fixes #3192
Fixes #3145
Fixes #3024

Don't use touch events anymore as everything works fine with `mouseup/down`

Fixes ng-bootstrap#3446
Fixes ng-bootstrap#3437
Fixes ng-bootstrap#3412
Fixes ng-bootstrap#3192
Fixes ng-bootstrap#3145
Fixes ng-bootstrap#3024
@codecov-io
Copy link

Codecov Report

Merging #3454 into master will increase coverage by 0.07%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3454      +/-   ##
==========================================
+ Coverage    91.1%   91.17%   +0.07%     
==========================================
  Files          95       95              
  Lines        2787     2787              
  Branches      519      516       -3     
==========================================
+ Hits         2539     2541       +2     
  Misses        189      189              
+ Partials       59       57       -2
Flag Coverage Δ
#e2e 55.61% <85.71%> (+0.07%) ⬆️
#unit 88.29% <57.14%> (+0.07%) ⬆️
Impacted Files Coverage Δ
src/util/autoclose.ts 92% <85.71%> (+8%) ⬆️

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 d3f44ca...f8672e3. Read the comment docs.

Copy link
Contributor

@peterblazejewicz peterblazejewicz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@maxokorokov maxokorokov merged commit c08e3b4 into ng-bootstrap:master Nov 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment