TC reloads quick view after Tabbing back to VP filelist

Bug reports will be moved here when the described bug has been fixed

Moderators: white, Hacker, petermad, Stefan2

Post Reply
User avatar
MVV
Power Member
Power Member
Posts: 8702
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

TC reloads quick view after Tabbing back to VP filelist

Post by *MVV »

Tested with Virtual Panel.

1. Add some large JPG file to VP (at least 10-14 MPx, bigger is better), open quick view.
2. Use Tab to jump to quick view panel.
3. Fit image to window and zoom image in (I'm using Imagine).
4. Use Tab to return back to VP panel. You will notice that quick view panel shows image that is fit to window again - TC reloaded an image.

It can't be reproduced with regular file panel. Also it can't be reproduced with old Temporary Panel plugin (which doesn't support FsLinksToLocalFiles).
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48005
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

I think that it's done because the link target may have changed - changing that may have some unwanted side effects...
Author of Total Commander
https://www.ghisler.com
User avatar
MVV
Power Member
Power Member
Posts: 8702
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

Christian,

TC doesn't reload panel contents (doesn't ask for file attributes) - so it doesn't check if time/date/size changes. Is it really necessary to update file only on the off-chance?

Maybe you should check if file date/time/size have changed?.. Also you may check if link's target (or its attributes) have changed and reload file if it is so (but not always). Or maybe it is better to allow user to update it manually? I think such cases are really rare.

Also I think there is no reason to reread the file if VP tells that all is OK - it is a VP problem to report file change.
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48005
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

It's unfortunately not so easy to change, so for now I will leave it as it is, sorry.
Author of Total Commander
https://www.ghisler.com
User avatar
MVV
Power Member
Power Member
Posts: 8702
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

But you may disable it, it's quite easy, right? :)

I think most people who use Tab in QV window just want to finish viewing this file, not to update QV.
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48005
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

I would do it if it were that easy, but I get an OnChange event (or is it OnClick) when changing panels even if the file doesn't change. I would pobably introduce 20 new bugs if I just checked for changed line number...
Author of Total Commander
https://www.ghisler.com
radeknews
Junior Member
Junior Member
Posts: 15
Joined: 2005-10-27, 13:51 UTC
Location: Switzerland

Post by *radeknews »

Gents,

the file in the quick-view seems to reload upon just about any keystroke, (e.g. space or even num-lock and caps-lock), as well as upon a mouse-click in the source panel (below, with a short file list), so this is not just at changing panels.

I appreciate you both taking a look into this.

Best regards
Radek
User avatar
MVV
Power Member
Power Member
Posts: 8702
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

Wow, it is fixed in TC8b16. :D
11.01.12 Fixed: Quick view (Ctrl+Q) in temporary panel: Only reload preview if the link target actually changed (32/64)
Thanks, Christian!
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48005
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

Thanks for checking! I didn't have too many bugfixes to do this week, so I found the time to test this in detail.
Author of Total Commander
https://www.ghisler.com
Post Reply