Bug 260844

Summary:	url bar does not update correctly
Product:	[Unmaintained] rekonq	Reporter:	Felix Rohrbach <fxrh>
Component:	general	Assignee:	Andrea Diamantini <adjam7>
Status:	RESOLVED UPSTREAM
Severity:	normal
Priority:	NOR
Version:	latest git snapshot
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Felix Rohrbach 2010-12-21 00:27:30 UTC

Version:           latest git snapshot (using KDE 4.5.4) 
OS:                Linux

On some sites, if you click on a link and then press the back button immediately, the url bar still shows the url of the link you clicked on while the page before is shown.

Reproducible: Always

Steps to Reproduce:
1. Go to http://doc.trolltech.com/4.6/
2. Click on the link "New in Qt 4.6"
3. Now, while the new page is loading, press the back button

Actual Results:  
Url bar shows "http://doc.trolltech.com/4.6/qt4-6-intro.html"

Expected Results:  
url bar should show "http://doc.trolltech.com/4.6/"

Comment 1 Felix Rohrbach 2011-01-21 17:31:11 UTC

May be related to bug #246367

Comment 2 Andrea Diamantini 2011-01-22 10:05:43 UTC

probably yes. My first toughts are about a QtWebKit bug. But I'm not sure. We basically update urlbar on QWebView::titleChanged() signal emitted. Here the page loading should be stopped by the history()->backItem() call and previous page reloaded from cache.
If this is true (again, not sure), a first trivial fix can be just update history buttons on loadFinished, so that user no more can click back while site is loading :)

Needs investigation.

Comment 3 Andrea Diamantini 2011-03-10 23:53:12 UTC

it should be fixed now. rekonq 0.6.85. Can you confirm it?

Comment 4 Felix Rohrbach 2011-03-11 00:34:45 UTC

It's sort of fixed, but I have to say that I don't like the fix at all.

Firstly, this fix just applies if you use the buttons to go back. If you use backspace (or my vertical mouse wheel navigation) the bug still exists.

Secondly, I don't like to wait for a page to be loaded fully before I can move again in history. I think there are a lot of use cases where you don't want to wait, e.g. you go back once, but while the page appears you realise that you need to go back twice. Now there's a really big picture from a slow server on this page and you have to wait the whole time until it's loaded.
This patch really slows down the work flow, especially for power users.

If you still want this workaround for 0.7, please don't fix the backspace/scroll wheel navigation. I guess it is mostly used by power users and they will have less problems with a wrong url bar than with a time consuming workaround.

Comment 5 Felix Rohrbach 2011-03-21 19:34:25 UTC

I can reproduce this with arora, so I guess it is upstream.

Comment 6 Andrea Diamantini 2011-12-16 22:33:28 UTC

Closing as resolved/upstream then. Thanks.