Addressline is overwritten when dropping URL

Bug #208742 reported by m_loginname
6
Affects Status Importance Assigned to Milestone
kdebase (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

This behavior can be reproduced in the following way:

1. Search Wikipedia with Google in a new konqueror session

Alt + F2 -> type gg:Wikipedia -> Return

2. Move over the first search result link (Wikipedia) and drag the URL right beside the Google Search tab.

Now the Wikipedia main page will be opened in the second tab, but the address for the first tab is overwritten, so the favicon of the addressline and the tab also.

Revision history for this message
m_loginname (soeken) wrote :

I am using kubuntu 7.10 with konqueror from ppa.launchpad.net.

I created a fix for the error. It affects the file kdebase-3.5.9/konqueror/konq_tabs.cc

487d486
< m_pViewManager->mainWindow()->openURL( newView, lstDragURLs.first(), QString::null );
488a488
> m_pViewManager->mainWindow()->openURL( newView, lstDragURLs.first(), QString::null );

Revision history for this message
Yuriy Kozlov (yuriy-kozlov) wrote :

Thank you for reporting this and working on the fix, but it looks to me like your patch doesn't do anything. Maybe you made a mistake generating the patch?

Anyway, I can't reproduce this in KDE 4.1 on Intrepid, so I am closing this as fixed.

Changed in kdebase:
importance: Undecided → Low
status: New → Fix Released
Revision history for this message
m_loginname (soeken) wrote :

Yes, the diff looks very strange. But it moves the line m_pViewManager->mainWindow()... below the following line. I can also confirm that it is fixed in konqueror4

Revision history for this message
Yuriy Kozlov (yuriy-kozlov) wrote :

Ah, I see. In the future, please use diff -u, it's a little more commonplace (in my experience) and easier to read.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.