It was added here:

commit a55ef906ae595065a3b3dcaf2f9dd1cb2fd4b7b2
Author: Csaba Karai <cskarai@freemail.hu>, Mon Apr 13 22:11:06 2009 +0000 (9 years ago)
Committer: Csaba Karai <cskarai@freemail.hu>, Mon Apr 13 22:11:06 2009 +0000 (9 years ago)
Follows: v2.0.0
Precedes: v2.1.0-beta1
Branches: <Expand>

FIXED: [ 2609483 ] unpacking to directories with international chars +
interview fixes

Do you know if we can access old issues by a number?

One of the use cases I can imagine is copy small files to the folder so that new items appear before current item - so the current item will be pushed down and become invisible quickly. Just an idea, I haven't tested it yet.

Thanks for taking the time to trace the code changes!

But I'm not sure we are talking about the same issue. The current bug report (392218) has a video attachment which shows the problem perfectly. I'm also facing the problem daily, I was just lazy to look into it :).

The issue was created by FileSystem rework of Alex (which was an awesome step forward) from last year I think. The item list is differently updated since - changed item is added and removed in model. This was done differently before so it did not trigger the makeItemVisible calls.

Anyway the use-case is simpler. You have long enough item list so you can scroll. Focus an item and scroll so that the focused item is outside the viewport. Currently you are not able to do that if any of the items is constantly updating (e.g. when a file download is in progress).

We are talking about the same issue and I also experience it from time to time. I was replying to this:

I don't know about a use-case that might be broken by this change.

as I found the change that's introduced the lines you want to remove. As the lines were added as a response to a bug, I was interested what was the problem. Although, 9 years has passed, it might be not relevant anymore.

The code change seems fine. I'll try to test this patch in the next few days.

I found the change that's introduced the lines you want to remove. As the lines were added as a response to a bug, I was interested what was the problem.

Sorry, my bad, I understand now! Unfortunately I also don't know how to find the antient bug reports.

One of the use cases I can imagine is copy small files to the folder so that new items appear before current item - so the current item will be pushed down and become invisible quickly. Just an idea, I haven't tested it yet.

This one I just tested and current item kept being visible (view was appropriately scrolled).

I still have the issue if the currently updating item is focused. But that can also be a feature for somebody, I'm not sure. Anyway this diff seems to me as an improvement over current situation. That is if we don't find any other issues of course :).

In D11699#235693, @martinkostolny wrote:

I still have the issue if the currently updating item is focused. But that can also be a feature for somebody, I'm not sure. Anyway this diff seems to me as an improvement over current situation. That is if we don't find any other issues of course :).

Thanks for mentioning. I'll try to pay attention to this during testing.

I also confirm the issue when current item is being updated. Can't scroll with mouse either. Interestingly, it's also the case when the very next to the item, which is being updated, is current - also can't scroll.