[bisq-network/bisq] Fix popup window ordering problem (#4360)

James Cox notifications at github.com
Tue Jul 7 21:53:19 UTC 2020


There is a race condition in PopupManager when with a stack of 2 popups, the top popup is being replaced by another top-level popup. The base layer popup ends up on top of the z-order, obscuring the correct one.

The race condition in PopupManager was within the onHidden() method which invokes the next window in the queue.  The call to displayNext() needs to be asynchronous so that a newer popup can take precedence.

Fixes #4293

You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/4360

-- Commit Summary --

  * Fix popup window ordering problem

-- File Changes --

    M desktop/src/main/java/bisq/desktop/main/overlays/popups/PopupManager.java (5)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/4360.patch
https://github.com/bisq-network/bisq/pull/4360.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4360
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200707/13a22177/attachment.html>


More information about the bisq-github mailing list