This happened in v1.8.3 I click on an item and the tree view scrolls so the item is out of the viewport. What seems to happen is: if I have a directory structure like /a /b /c /d /e the tree view seems to be trying to keep /a in the view, even tho I click on "e". For this to happen, there needs to be enough "distance" between a and e that they don't both fit on the visible area.
Created attachment 8324 thunar-tree-view.gif I can't reproduce with 1.8.4, see the attached gif. Am I missing something? Commands to create the folders and open the last one: mkdir -p a/b/c/d/e/f/g/h/i/j/k/l/m/n/o/p/q/r/s/t/u/v/x/y/w/z thunar a/b/c/d/e/f/g/h/i/j/k/l/m/n/o/p/q/r/s/t/u/v/x/y/w/z
In your example, if I selected folder "z" and made sure the tree view was scrolled so I could see "z", then I clicked on "w", it would select "w", but then scroll so "a" was visible, and "w" was out of view. If you don't see that, I'll try to make a little video or gif. Thanks for responding
Created attachment 8327 little script to make the test directories
https://drive.google.com/open?id=1npuDNiNrnU0BoF9wYsUCeigMj9Bywyz8 video showing what I see
Ok, I see, I can reproduce the problem. Thanks for reporting ! I think the "auto scroll" should not be used at all, if any item in the tree is clicked. IMO auto-scroll on the tree only should be used if the right-side folder changes.
*** Bug 15194 has been marked as a duplicate of this bug. ***
Created attachment 8339 patch fixing the bug It seems like "gtk_tree_view_set_cursor" should not be executed with high frequency. The attached patch fixes the problem by always stroring the latest path and only execute "gtk_tree_view_set_cursor" once. Please take a try for the patch and let me know if it fixes the problem for you / if there possibly are any side-effects.
(In reply to alexxcons from comment #7) > Created attachment 8339 > patch fixing the bug error: corrupt patch at line 48 After copying and pasting, the patch seems to fix the issue.
Thanks for testing ! > error: corrupt patch at line 48 Meh .. just found out that I can create patches via eclipse .. seems like I should better keep using the git format-patch in the console :)
Alexander Schwinn referenced this bugreport in commit 0735e5bb8dd3b8fbac13f1b4165e30e27cea643f tree view jumps (scrolls) when clicking on a directory (Bug #15174) https://git.xfce.org/xfce/thunar/commit?id=0735e5bb8dd3b8fbac13f1b4165e30e27cea643f
Alexander Schwinn referenced this bugreport in commit c0d13b6a806cd25c43fe75aadf7936417a0109b8 tree view jumps (scrolls) when clicking on a directory (Bug #15174) https://git.xfce.org/xfce/thunar/commit?id=c0d13b6a806cd25c43fe75aadf7936417a0109b8
Alexander Schwinn referenced this bugreport in commit 99c08e8ba9216d9f7e330d68168b0d8f047ac426 tree view jumps (scrolls) when clicking on a directory (Bug #15174) https://git.xfce.org/xfce/thunar/commit?id=99c08e8ba9216d9f7e330d68168b0d8f047ac426
Fixed for master , 4.14 and 4.12 branch. To be released in 1.8.5 and 1.6.17
https://drive.google.com/open?id=1jsmOdu7tok0igUuX3vDdxiba7SsKwm1E Still seeing some weirdness I click down the tree as shown, when I get to zzzzzz it jumped back to xxxxxx - always
meh ... should have used your example for testing instead of my own. Will have another look at it when I have time
Sorry, I cannot reproduce this extra-weirdness .. you are sure you are using thunar compiled from master / 4.14 branch ? You need to quit the system-thunar daemon .. best use ./thunar -q;./thunar to do so.
I grabbed the master from https://github.com/xfce-mirror/thunar I saw the 0735e5b commit and the change seemed to be in there So I should use https://github.com/xfce-mirror/thunar/tree/xfce-4.14 ?
Both, master and 4.14 branch are fine, I patched both. I just wanted to make sure that you quit the currently running thunar daemon for testing. Because if an old thunar daemon is running in the background, you cannot start the newly build thunar. ( Instead the old daemon will spawn a window)
Created attachment 8346 build error message Yes, I've been doing thunar --quit. I do have a little trouble with build, but at the examples and plugins stage. Do you know how to fix this. I think the plugins problem is the same.
Not sure which build commands you used, and why you have " thunar-1.8.2" stuff there.. looks suspicious. Here what should work: git clone https://git.xfce.org/xfce/thunar cd thunar ./autogen.sh <at this stage you will need to install missing development packages, until autogen.sh runs fine> make to run it from source: cd thuner ./thunar -q;./thunar ... and if you want to install it: sudo make install
1.8.2 is there simply because I was rebuilding 1.8.2 as it doesn't have this scroll issue
So can you please take a try with master (or the 4.14 branch), using the steps above ?
That is what I did to make the video in https://bugzilla.xfce.org/show_bug.cgi?id=15174#c14 As mentioned, I do have trouble with the build when I get to examples/plugins, that I can't figure out. But I don't think that affects the thunar behaviour in question. The autogen seems OK Build Configuration: * GIO UNIX features: yes * GUDev (required for thunar-volman): yes * Mount notification support: yes * Debug Support: yes * GObject Instrospection support: yes Additional Plugins: * Advanced Properties: yes * Simple Builtin Renamers: yes * Trash Panel Applet: yes * User Customizable Actions: yes * Wallpaper support: yes I don't know how to go any further at this point.
*** Bug 15313 has been marked as a duplicate of this bug. ***
The fix introduced a new bug. Selecting the "Desktop" or "Trash" entries in the tree view will prevent further selections with the mouse cursor.
Mukundan has posted some builds for f29 and f30 which fix this awful bug: https://bugzilla.redhat.com/show_bug.cgi?id=1701829 for F29 - https://koji.fedoraproject.org/koji/taskinfo?taskID=34349581 for F30 - https://koji.fedoraproject.org/koji/taskinfo?taskID=34349616 For anyone who has a little trouble finding where the packages are, like i did, if you click on the descendants/build link for your arch, you'll be taken to the goodies. And for anyone else trying to reproduce this bug, eg on the buggy 1.8.4-1.f29, just: 1) make sure you expand a bunch of folders so they don't fit in the tree/folder pane, and a scrollbar appears 2) scroll down in the tree view. you wont see te bug if yo u let excess folders go off the bottom of the pane while the top is still viewing the parent of the subfolders you're clicking on 3) to see it at its worst, make sure you have a folder with many subfolders in it, and scroll down so the parent folder is not visible in the tree. 4) select a folder, and watch i horror as the whole tree view scrolls up to bring the parent closer into view, whilst the folder you selected goes down, and maybe off the bottom of the screen even. Pls ensure the fix gets upstreamed ASAP!! If i never see this bug again, i will be extremely grateful. I am often a bastard in these bugzillas, because many devs don't seem to give a crap. They want to produce shiny garbage and don't care if noone can actually use it. I've seen first hand in many bugzillas that xfce devs are different, they know they're working on a product people actually use, and they strive to produce a product that lets people get the job done. So, thank you to all xfce devs, you are effing awsm, pls keep up the good works!
(In reply to Theo Linkspfeifer from comment #25) > The fix introduced a new bug. Selecting the "Desktop" or "Trash" entries in > the tree view will prevent further selections with the mouse cursor. Theo is right. Damn. With the packages built by Mukundan, Desktop is ok. I can click on that, and it seems ok. But if i click on Trash, it's game over, i can't select anything else :-( Nevertheless, you guys must be very close to fixing this, so, I salute you, and will eagerly await a solid fix.
> The fix introduced a new bug. Selecting the "Desktop" or "Trash" entries in > the tree view will prevent further selections with the mouse cursor. This one I can reproduce. For Johns reproducer I have to try later ... will see what I can do about it. @John Well, sometime it is like that, fix one thing, something else breaks :[
Created attachment 8454 patch II Ok, I completly reworked & further simplified this goddam complex method. Now it looks like the bug is fixed: - I tried to make it fail with both reproducers mentioned here .. seems to be fine now. - Trash & Desktop can be accessed. - It is still possible to open a folde when it's parent has no read permission (Bug #14900) Please all of you, take a try for this patch (apply it on master!) and let me know if you can still see any wired behaviour !
With that patch everything seems to work properly now. Good job!
I just tried the new package built by Mukundan over on: https://bugzilla.redhat.com/show_bug.cgi?id=1701829 I have not tried Alex's new patch as I am not buildign from source but if Mukundan is using Alex's new patch, i think there is still one problem. The new package does fix the original scrolling issue, and also the Trash bug. But now, when i click on a folder in the tree view, it gets expanded in the tree view, if it has subfolders. That never used to happen before. Before, when you select a folder in the tree view, it opens in the details pane on the right, but the subfolders do NOT get expanded in the tree view. To expand a folder in the tree view, you had to click the arrow to the left of folder, which is how it should stay! If we can stop the tree view expanding when you click on a folder (which has subfolders) then things will be perfect. Many thanks!
Thanks alot , for testing ! > But now, when i click on a folder in the tree view, it gets expanded in the tree view, if it has subfolders. > That never used to happen before. I just took a try with thunar from the 4.12 branch .. this behavior actually worked in the old 4.12 thunar .. it just stoppen to worked after some of my tree-view fixes ;) IMO the auto-expand is nice. Removing it may not please other users. Please open a separate bug where we can discuss on that. ( Possibly we could add a xfconf setting to enable/disable it).
Alexander Schwinn referenced this bugreport in commit 25276cb404ce8a9961f00f5110f24b1e378908fc tree view jumps (scrolls) when clicking on a directory II (Bug #15174) https://git.xfce.org/xfce/thunar/commit?id=25276cb404ce8a9961f00f5110f24b1e378908fc
Alexander Schwinn referenced this bugreport in commit d8befdca1d8f4f148c33c8ed8b11a49804c79344 tree view jumps (scrolls) when clicking on a directory II (Bug #15174) https://git.xfce.org/xfce/thunar/commit?id=d8befdca1d8f4f148c33c8ed8b11a49804c79344
Alexander Schwinn referenced this bugreport in commit 5fdeb52e20f8cf7c493754ae09557fe3bc0377de tree view jumps (scrolls) when clicking on a directory II (Bug #15174) https://git.xfce.org/xfce/thunar/commit?id=5fdeb52e20f8cf7c493754ae09557fe3bc0377de
Ok, think the patch got sufficient testing. Patched master, 4.14 and 4.12 branch. To be released in 1.8.5 and 1.6.17
(In reply to alexxcons from comment #32) > IMO the auto-expand is nice. Removing it may not please other users. Please > open a separate bug where we can discuss on that. ( Possibly we could add a > xfconf setting to enable/disable it). I really disagree that the auto-expand is nice. It may seem like a good idea, but it really is not. When users click on a folder in the tree view, their intent is for the contents to appear in the right pane. Having the folder auto-expand in the tree view gets very tiring very quickly. It violates the principle of least surprise, and, again, makes using the tree view disorienting. I do not want to hold up Windows explorer as the standard Thunar should be aiming for... but, Windows Explorer does not auto-expand in tree view, and... neither should Thunar. But I'm happy for it to be discussed in a separate bug. Ta so much for fixing the scroll bug, that one was really painful :-)
(In reply to John from comment #37) > (In reply to alexxcons from comment #32) > > IMO the auto-expand is nice. Removing it may not please other users. Please > > open a separate bug where we can discuss on that. ( Possibly we could add a > > xfconf setting to enable/disable it). Oh, and if there can be an xconf setting for it, then that would be awesome, and keep everybody happy :-)
When and where will this patch be released? I'd like to push my distro to integrate that in order to get a usable file browser back finally.
> Patched master, 4.14 and 4.12 branch. To be released in 1.8.5 and 1.6.17 Latest on 19th of May, which is feature freeze for xfce4.14. Meanwhile, you have many options: 1) Compile from source 2) Ask your distro to patch the existing package 3) Just use bookmark view instead of tree-view till your distro ships the fix ;) Subscribe to xfce-announce, so you wont miss the release: https://mail.xfce.org/mailman/listinfo/