Bug 260844 - url bar does not update correctly
Summary: url bar does not update correctly
Status: RESOLVED UPSTREAM
Alias: None
Product: rekonq
Classification: Applications
Component: general (show other bugs)
Version: latest git snapshot
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Andrea Diamantini
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-21 00:27 UTC by Felix Rohrbach
Modified: 2011-12-16 22:33 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.