Web UI flicking to selection

[monkeyman23555]

Hello everyone

This is a trivial bug, I am simply making a ticket to ensure that it has been noted. (If the web UI is replaced by something other than clutch this will probably not be accurate anymore.)

When you select a torrent in the Web UI and scroll to a different location so that the selected torrent is out of view, then when the refresh happens the view flicks back to the selected torrent. This works in both normal and compact modes.

I tested it with both Firefox and Safari form my headless NAS running 2.12 and my mac transmission running 2.13.

[jordan]

This is unlikely to be fixed unless someone submits a patch.

The web client currently has no maintainer, and is getting by on submitted patches.

[fx]

Looks like its the scrollToElement function, Commenting line number transmission.js:461 seems to stop it without any adverse effects, Makes me wonder what is the purpose of that function...

This is where i made the change:

selectElement: function( e, doUpdate ) {
    e.addClass('selected');
    //this.scrollToElement( e );
    if( doUpdate )
        this.selectionChanged( );
},

[jordan]

I think that line was to ensure that the gui scrolls to any torrent that is selected by the app rather than by the user. (Any torrent seleced by the user will already be onscreen.)

However I can't find any code path for the selection-by-app case. If there was one, I don't see it now.

So maybe that line should be removed? I don't see any downside to removing it and getting wider user testing on this theory.

[jordan]

thanks, fx!

https://trac.transmissionbt.com/ticket/3820#comment:3

[livings124]

r12342

[Tenzer]

I think the offending line was intended for users navigating the web interface with the keyboard, since it makes the list scroll up or down when using the arrow keys on the keyboard to navigate with. With this line commented out this will no longer happen, and make it impossible to see any torrents not on in the current view, by only using the keyboard.

I don't know what the preference is here, and how highly the keyboard navigation is prioritized, but I am reopening this ticket to allow people here to find out.

Personally I haven't had issues with the flickering the issue originally concerned, so I would prefer if the line stayed in as in Transmission 2.22.

[bulljit]

@Tenzer

Try the attached patch on the latest version of the transmission.js (v2.30b4). It simply adds line 578:

tr.scrollToElement(tr._rows[i]); // see https://trac.transmissionbt.com/ticket/3820

This should resolve your keyboard navigation problem seen with r12342.

BTW the "flickering" issue is only seen when you advance the scrollbar past the currently selected torrent (using your mouse). I assume you only use your keyboard for navigation and therefore never elicited this bug.

[jordan]

looks right to me. Thanks buljit, thanks Tenzer :)

[jordan]

r12430: apply patch from buljit to fix keyboard navigation regression