Blueprint: Dismissing a Popover that is in a Dialog is closing the Dialog

[tgreen7]

Bug report

• __Package version(s)__: 2.0.0-rc.2
• __Browser and OS versions__: Chrome 64.0.3282.186, MacOS 10.13.3

test site: https://tgreen7.github.io/blueprint-popover/
test repo: https://github.com/tgreen7/blueprint-popover

[adidahiya]

This is expected, since the popover is outside the dialog, and outside clicks are allowed to close the dialog. You can get around this by disabling canOutsideClickClose on the Dialog or making the Popover usePortal={false}.

[tnrich]

@adidahiya this has only started happening in v2 of blueprint though. Was this change in behavior expected?

[giladgray]

@tnrich i'm not sure that's true--fairly sure this would happen in 1.0 too. we haven't touched the logic that closes on outside click.

[tgreen7]

@giladgray here is a test in 1.0 https://tgreen7.github.io/blueprint-popover-v1/
it seems to be working fine there. (repo https://github.com/tgreen7/blueprint-popover-v1)

[giladgray]

(incorrect close, wrong issue number in PR)

[giladgray]

other related issues exist