Grid view selection lost when trying to use the scroll bar

[teosarca]

[teosarca]

More, the selection is changed even if i am clicking somewhere in the page.... that shall be fixed too

[damianprzygodzki]

Scroll bar looks like a bug, or changes in the browser behavior. But selection is always disappearing when we click outside the table.

[cadavre]

"click outside the table" is always expected action. It is pattern also used everywhere else in metasfresh. It only shouldn't de-select on moving into Selection attributes or when pressing one of Quick actions (which is obvious).

But scrollbar shouldn't deselect – that's true.

[bambeusz]

It seems that in modern browsers scrollbar is not visible by default, it's shown only when the user scrolls (at least on OSX). For the browser clicking a few pixels from right edge, is equal to clicking on scrollbar. This makes detecting of click on scrollbar impossible, as it is just an area, very close to right side.

Another thing is that scrollbar of the whole content is not always the only scrollbar on the page. We'd need to detect clicking on scrollbar on every container.

imho, metasfresh is keyboard navigation friendly, isn't it? What's the point of scrolling down with mouse click, while there is far better way to do it. E.g. using arrows, or scrolling with mouse scroll.

wdyt

[teosarca]

Suggestion: as discussed last week, deselect the row only when clicking in the parent container.
I am open on any other suggestion which would have a similar effect (i.e. don't deselect on scrollbar).

@chmieludaniel

imho, metasfresh is keyboard navigation friendly, isn't it? What's the point of scrolling down with mouse click, while there is far better way to do it. E.g. using arrows, or scrolling with mouse scroll.

IMHO that would be a big usability drawback.
Scrollbars are also a conventional indicator that there is more to see...

[metas-ts]

@teosarca @metas-lc if you move a task to test or if you pick up a task for testing please check and make sure that the task is in the release notes.
i'm adding this one now

[metas-lc]

Where do i have to select so my line would get unselected?

[damianprzygodzki]

@Dunkat ?

[metas-lc]

was the fix integrated? I didn't notice any difference

[metas-lc]

if there was a way to unselect the lines, i would expect to click somewhere in the white space. Maybe i am wrong

[Dunkat]

something broke, maybe in another merge, need to check

[Dunkat]

Should work now. Just click outside table in the white space.

[metas-lc]

IT

• select the line and then select white space => line is deselected OK
• select the line and then change attributes => line is still selected OK
• select the line and then select the vertical scrollbar => line is deselected NOK
• select the line and then select the horizontal scrollbar => line is deselected NOK

[metas-mk]

Tested. Works nicely as required.
Retested these cases:

• select the line and then select the vertical scrollbar => line stays selected OK
• select the line and then select the horizontal scrollbar => line stays selected OK