[PATCH] JS Form Popups

[Ralf Hoelzer]

> Thanks, applied.
> However, 2 seconds after applying I had a thought: doesn't this patch 
> mean that people who use the "smart" policy will simply see their form NOT
> submitted, with no explanation why, if it would pop up another window? I
> wonder if this is the best choice - I mean, although I don't like popups, if
> I have to send some information, I have to do it, even if it will pop up a
> window.... Maybe we could post the form to the current window (clearing
> "target"), but this hack might break some sites of course....      

You're right, "smart" might break some sites this way. Actually, 
isWindowOpenAllowed() which is used for the window.open() checks works great 
for form submits as well. I added a call to it,  if the policy is "smart". I 
tried this on a testcase and it seems to work nicely.

Some kind of feedback that a window has been blocked, like the status bar icon 
in Mozilla Firebird, might be a nice feature.

The new patch is attached.

regards,

Ralf


-------------- next part --------------
A non-text attachment was scrubbed...
Name: form3.patch
Type: text/x-diff
Size: 737 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20030606/70ad2fb8/attachment.patch>