This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 269456 - JavaFX Presenter does not handle popups
Summary: JavaFX Presenter does not handle popups
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Html4j (show other bugs)
Version: 8.2
Hardware: PC Mac OS X
: P3 normal (vote)
Assignee: Jaroslav Tulach
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-12-30 13:34 UTC by maxnitribitt
Modified: 2017-02-03 17:32 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
patch that opens popup when requested (16.77 KB, patch)
2016-12-30 13:41 UTC, maxnitribitt
Details | Diff
updated patch (1.65 KB, patch)
2016-12-30 13:56 UTC, maxnitribitt
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description maxnitribitt 2016-12-30 13:34:03 UTC
Behaviour of html4J presenters is inconsistent on different platforms. With Browser Presenter we can open a new Window from html like this:

<script>
function openSecondaryWindow() {
    return win2 = window.open('index_1.html', 'secondary', 'width=300,height=150');
}
</script>
<button onclick="openSecondaryWindow();">Open</button>

With JavaFX WebView Presenter the index_1.html will be opened in the same window replacing the existing content.
Comment 1 maxnitribitt 2016-12-30 13:41:11 UTC
Created attachment 163311 [details]
patch that opens popup when requested
Comment 2 maxnitribitt 2016-12-30 13:43:19 UTC
Sorry, it seems that my sources are outdated so there's some rubbish in the patch.
Comment 3 maxnitribitt 2016-12-30 13:56:21 UTC
Created attachment 163313 [details]
updated patch
Comment 4 Jaroslav Tulach 2017-02-03 17:32:05 UTC
commit eef06bba0675fc4535eb5c68ff73b0a30b303d31
Author: Jaroslav Tulach <jtulach@netbeans.org>
Date:   Fri Feb 3 18:30:30 2017 +0100

    #269456: window.open in JavaFX presenter shows new popup window


Thanks for the patch. The change will be included in version 1.4