Column Resorter: Error thrown upon changing order

Feedback & Ideas
1

Hi,

I noticed each time columns are sorted using the arrows a console error is thrown.

image
image1791Ɨ427 60.1 KB

2024-10-23 16:03:46,502 ERROR CEFNodeView Uncaught (in promise) TypeError: ee.scrollUpIntoView is not a function (source: http://org.knime.core.ui.dialog/uiext/defaultdialog/NodeDialog.js; line: 54082)
2024-10-23 16:03:46,649 ERROR CEFNodeView Uncaught (in promise) TypeError: ee.scrollUpIntoView is not a function (source: http://org.knime.core.ui.dialog/uiext/defaultdialog/NodeDialog.js; line: 54082)
2024-10-23 16:03:46,772 ERROR CEFNodeView Uncaught (in promise) TypeError: ee.scrollUpIntoView is not a function (source: http://org.knime.core.ui.dialog/uiext/defaultdialog/NodeDialog.js; line: 54082)
2024-10-23 16:03:46,912 ERROR CEFNodeView Uncaught (in promise) TypeError: ee.scrollUpIntoView is not a function (source: http://org.knime.core.ui.dialog/uiext/defaultdialog/NodeDialog.js; line: 54082)
2024-10-23 16:04:15,089 ERROR CEFNodeView Uncaught (in promise) TypeError: ee.scrollUpIntoView is not a function (source: http://org.knime.core.ui.dialog/uiext/defaultdialog/NodeDialog.js; line: 54082)
2024-10-23 16:04:21,615 ERROR CEFNodeView Uncaught (in promise) TypeError: me.scrollDownIntoView is not a function (source: http://org.knime.core.ui.dialog/uiext/defaultdialog/NodeDialog.js; line: 54082)
2024-10-23 16:04:22,392 ERROR CEFNodeView Uncaught (in promise) TypeError: ee.scrollUpIntoView is not a function (source: http://org.knime.core.ui.dialog/uiext/defaultdialog/NodeDialog.js; line: 54082)

On another note, it still is impossible, or tā€™s a regression, to delete not found column. A full reset is required instead.

Both are less critical but contribute to a perceived inferior app quality / degradation.

Best
Mike

2

Hey @mwiegand,

I have made a ticket on your main concern previously from thread below although I just verified it still comes up with the use of arrow keys or just clicking up or down so I will also attach that to the ticket:

It used to be Ticket AP-23262, it was updated to UIEXT 2143

On your side note, did you mean hitting the ā€˜Reset Allā€™ button in the node?

Thanks for reporting!
TL

1 Like
3

Hi @thor_landstrom,

thanks aa low. Yes, I meant to push ā€œReset allā€ as the sole option to reset. I recall that I provided extensive feedback, including a mockup, once. Maybe during the community hacking days. I cannot find that anymore.

Best
Mike

1 Like
4

I am assuming the issue you are pointing is that you cannot just double click on it and have it disappear (which is how it behaves for other nodes). I will look into it to see if we have a ticket up documenting that behavior.

TL

5

Double click isnā€™t quite intuitive to remove soemthing but more to open or interact with something that exists. Iā€™d expect a button like an X either being displayed once hovering or permanently displayed right next to the already present buttons but only becoming enabled once a deletable column got selected. The first Iā€™d prefer as itā€™s the most intuitive and least obstructive.

1 Like