Bug 253708

Cannot reproduce this issue. Do you have an example site for this ? Clicking on a link that is supposed to open in a new unamed window, i.e. target=_blank works fine here.

Here is a test page for links. Click on the link that says "Open

Here is an example of a page using target=_blank:
http://www.w3schools.com/html/html_links.asp

Scroll down to the target attribute example, and click "try yourself"

Note: This is using kdewebkit inside konqueror, with default user agent (hm, should that change automatically when using kdewebkit??), in kde 4.5.2, using qt 4.7. Konqueror is configured to use new tab instead of new window. The default view is KHTML, so to test you need to activate view->kdewebkit.

ANY link using this (and possibly other new, named targets, dunno) fails, for example clicking in a youtube video to view it at the youtube site. That is an embedded youtube video at the end of this page:
http://minbaad.dk/sejl/archive/2010/17/october/article/myralf-og-hviid-vinder-vm-i-rc44/ - click "youtube" in the lower right corner of the video to get an empty tab - this is actually very annoying, since that URL can not be dragged to the new tab, as with HTML links...

On Tue, Oct 19, 2010 at 12:24 PM, Anders Lund <anders@alweb.dk> wrote:
> https://bugs.kde.org/show_bug.cgi?id=253708
>
>
>
>
>
> --- Comment #2 from Anders Lund <anders alweb dk>  2010-10-19 18:24:23 ---
> Here is an example of a page using target=_blank:
> http://www.w3schools.com/html/html_links.asp
>
> Scroll down to the target attribute example, and click "try yourself"
>
> Note: This is using kdewebkit inside konqueror, with default user agent (hm,
> should that change automatically when using kdewebkit??), in kde 4.5.2, using
> qt 4.7. Konqueror is configured to use new tab instead of new window. The
> default view is KHTML, so to test you need to activate view->kdewebkit.
>
> ANY link using this (and possibly other new, named targets, dunno) fails, for
> example clicking in a youtube video to view it at the youtube site. That is an
> embedded youtube video at the end of this page:
> http://minbaad.dk/sejl/archive/2010/17/october/article/myralf-og-hviid-vinder-vm-i-rc44/
> - click "youtube" in the lower right corner of the video to get an empty tab -
> this is actually very annoying, since that URL can not be dragged to the new
> tab, as with HTML links...

Ahhh... I now see the problem. It only occurs when you change the
rendering engine using View->Mode->WebKit! Because we do not handle
the case where the newly created part is not a KWebKitPart!

This is simply the same problem and it is a usage issue. I never use
both rendering engines at the same time ; so I do not know how
switching engines using View->Mode behaves. Anyhow, the issue is
simple enough to address ; so it should be fixed shortly....

SVN commit 1187648 by adawit:

- When a user clicks on a link that opens in a new window/tab, make sure the url is opened
  even when the returned KPart is not a KWebKitPart... 

CCBUG:253708


 M  +9 -1      webpage.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1187648

Fixed per comment #4 and back ported the fix to the stable kwebkitpart branch as well. See http://websvn.kde.org/branches/stable/extragear-kde4/base/kwebkitpart/

(In reply to comment #4)
> SVN commit 1187648 by adawit:
> 
> - When a user clicks on a link that opens in a new window/tab, make sure the
> url is opened
>   even when the returned KPart is not a KWebKitPart... 
> 
> CCBUG:253708
> 
> 
>  M  +9 -1      webpage.cpp  
> 
> 
> WebSVN link: http://websvn.kde.org/?view=rev&revision=1187648

Pages are opened in new tab with this fix.
But there is about:blank in address field and if I press on refresh page then blank page will be displayed.

SVN commit 1202466 by adawit:

- Better workaround for BR# 253708 so that the issue mentioned in comment #6
  of that report is handled properly.
- Set a dynamic QObject property named NavigationTypeUrlEntered whenever openUrl 
  is invoked so that we can distinguish user entered requests from other types.
- Reduced nested if statements whereever possible.
- Enclosed literal text with QLatin1String.

CCBUG:253708


 M  +48 -46    webpage.cpp  
 M  +2 -0      webview.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1202466

SVN commit 1205667 by adawit:

- Yet more createWindow related changes. Guess spoke too soon on previous commit.
  This implementation is a robust generic fix that handles all window creation
  requests from the browsing engine, i.e. QWebPage::createWindow. 

  Besides dealing with previous bugs such as 253708 and 258367 more appropriately, 
  it also prevents the duplicate creation of a new window if another one with the 
  same name already exists.

CCBUG:253708
CCBUG:258367


 M  +0 -1      CMakeLists.txt  
 D             fakepage.cpp  
 D             fakepage.h  
 M  +171 -127  webpage.cpp  
 M  +36 -0     webpage.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1205667