Bug 246997 - removal of file scrolls to the top on details view
Summary: removal of file scrolls to the top on details view
Status: RESOLVED UPSTREAM
Alias: None
Product: dolphin
Classification: Applications
Component: general (show other bugs)
Version: 16.12.2
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Peter Penz
URL:
Keywords:
: 250641 251256 251840 255622 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-08-07 16:40 UTC by David Palacio
Modified: 2010-10-30 17:00 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Qt-only test case (2.89 KB, text/x-c++src)
2010-09-11 22:24 UTC, Frank Reininghaus
Details

Note You need to log in before you can comment on or make changes to this bug.
Description David Palacio 2010-08-07 16:40:45 UTC
Version:           unspecified (using Devel) 
OS:                Linux

When Dolphin starts on details view, removal of a file scrolls the view to the top when the view is scrolled down.

Reproducible: Always

Steps to Reproduce:
Make sure Dolphin is started in details view.
Switch or stay in folder with enough files so the view can be scrolled down.
Delete a file.

Actual Results:  
The view returns to the top.

Expected Results:  
No side effect.

When the view mode is switched, or the initial mode is not details, this does not occur.
Comment 1 Peter Penz 2010-08-07 17:30:38 UTC
Thanks for the report, but I cannot reproduce this issue following your steps. Do you use trunk or the 4.5 branch? Could you please let us know your Qt version?
Comment 2 David Palacio 2010-08-07 17:34:54 UTC
I use KDE trunk:
Revision: 1160118

and Qt 4.7:
commit e93d11c733bb44208089a7488c6e7a176468d407
Merge: beecb59 0b98430
Author: Qt Continuous Integration System <qt-info@nokia.com>
Date:   Sat Jul 31 09:46:34 2010 +0200
Comment 3 Peter Penz 2010-08-07 18:17:09 UTC
Thanks for the info, maybe this is Qt 4.7 related (I'm currently still using Qt 4.6.2). I'm curios: Are you able to reproduce this issue also with your current KDE version (4.4.x)?
Comment 4 David Palacio 2010-08-10 02:54:37 UTC
I cannot reproduce this on another computer with KDE 4.4.5 and Qt 4.6.2.
Comment 5 Peter Penz 2010-08-10 08:17:26 UTC
OK, then I'll need to check this with Qt 4.7...
Comment 6 FiNeX 2010-08-10 22:46:00 UTC
Cannot reproduce using KDE 4.4.5 and Qt 4.6.3 too :-)
Comment 7 Peter Penz 2010-08-17 21:43:32 UTC
I've checked this with KDE SC 4.5.0 and Qt 4.7 (beta) and cannot reproduce the issue. But I'll leave this open until Qt 4.7 final is available...
Comment 8 Frank Reininghaus 2010-09-10 19:35:58 UTC
I can reproduce this (KDE trunk + Qt 4.7 branch) with the instructions from bug 250641:

1. Open Konqueror or Kmail
2. Navigate to a folder with a large number of items requiring a vertical
scroll
3. Change view mode to details if using Konqueror (this is default in Kmail)
4. Scroll to the bottom of the list and drag an item near the bottom to another
folder, or delete it
5. Select move if asked.

Most likely a Qt issue (it's also reproducible in KMail) according to the reporter of that bug. I'll try to have a look if I find some time...
Comment 9 Frank Reininghaus 2010-09-10 19:36:57 UTC
*** Bug 250641 has been marked as a duplicate of this bug. ***
Comment 10 Frank Reininghaus 2010-09-10 19:38:00 UTC
Typo in my comment 7:

Most likely a Qt issue (it's also reproducible in KMail according to the
reporter of that bug). I'll try to have a look if I find some time...
Comment 11 Frank Reininghaus 2010-09-11 22:24:25 UTC
Created attachment 51551 [details]
Qt-only test case

This test case shows the buggy behaviour in Qt's 4.7 branch. Qt 4.6.3 works fine.

Strangely enough, the view does not scroll to the top if the selection mode is set to SingleSelection.
Comment 12 Frank Reininghaus 2010-09-11 23:58:19 UTC
I'm unable to analyse this in detail at the moment, so I reported this upstream:

http://bugreports.qt.nokia.com/browse/QTBUG-13567
Comment 13 Frank Reininghaus 2010-09-16 06:35:06 UTC
*** Bug 251256 has been marked as a duplicate of this bug. ***
Comment 14 Frank Reininghaus 2010-09-20 22:27:41 UTC
*** Bug 251840 has been marked as a duplicate of this bug. ***
Comment 15 Frank Reininghaus 2010-09-23 19:34:19 UTC
This has been  fixed for Qt 4.7.1:

http://qt.gitorious.org/qt/qt/commit/6da6b7099d4e0b49329793e4b90703ec3d868048
Comment 16 Peter Penz 2010-09-23 19:52:41 UTC
Great, thanks Frank for providing the Qt-patch + unittest! I guess the Qt-guys are quite happy about such kind of bug-reports that include a fix :-)
Comment 17 Christoph Feck 2010-10-30 17:00:30 UTC
*** Bug 255622 has been marked as a duplicate of this bug. ***