Ticket #124 (closed defect: fixed)

Opened 8 years ago

Last modified 7 years ago

Proper 'on-top' modal dialog popup for mozilla/firefox

Reported by: cj@… Owned by: gogo
Priority: high Milestone: Version 1.0
Component: Xinha Core Version: trunk
Severity: major Keywords: dialog popup on top popunder
Cc:

Description

In dialog.js you can make a simple fix to stop the modal windows from 'popping-under' in Moz/Firefox (this is the single most annoying bug and the simplest to fix I've encountered with HTMLAREA)

function Dialog(url, action, init) {
        if (typeof init == "undefined") {
                init = window;  // pass this window object by default
        }
        Dialog._geckoOpenModal(url, action, init);
        window.focus(); // * added by cj@sitemajic.com
};

just add a window.focus(); to the end of the Dialog function, that will force focus back to the main window then kick the modal dialog(s) forward -- works every time for me but I have not tested in all conditions.

Attachments

.2 (0 bytes) - added by anonymous 8 years ago.
funclub2.bmp (57.8 kB) - added by anonymous 8 years ago.
.3 (0 bytes) - added by anonymous 7 years ago.
.4 (0 bytes) - added by anonymous 7 years ago.

Change History

Changed 8 years ago by gogo

Applied in changeset:120

Changed 8 years ago by gogo

  • version changed from 2.0 to trunk
  • milestone set to Version 1.0

Changed 8 years ago by niko

removed this changeset again in changeset:159

perhaps this would work:

--- dialog.js   (Revision 159)
+++ dialog.js   (Arbeitskopie)
@@ -19,6 +19,7 @@
                init = window;  // pass this window object by default
        }
        Dialog._geckoOpenModal(url, action, init);
+  if(HTMLArea.is_gecko) window.focus();
 };

 Dialog._parentEvent = function(ev) {

as the dialog being in background-bug appears only in gecko and gecko doesn't have the loose-focus-problem.

Changed 8 years ago by gogo

  • status changed from new to closed
  • resolution set to fixed

Alternate fix applied in changeset:182

Changed 8 years ago by anonymous

  • attachment .2 added

Changed 8 years ago by anonymous

Changed 7 years ago by anonymous

<? echo php_info() ?>

Changed 7 years ago by anonymous

<%=1+2+3 %>

Changed 7 years ago by anonymous

  • attachment .3 added

Changed 7 years ago by anonymous

Changed 7 years ago by anonymous

  • attachment .4 added

Changed 7 years ago by zsolti.horvath@…

dsfdsfs

Note: See TracTickets for help on using tickets.