fskorgen commented on PR #8530: URL: https://github.com/apache/hop/pull/8530#issuecomment-5814911478
Good points — the second one convinced me. One correction on the first: the editor is still reachable. `Enter` and `F2` call `edit(...)` directly from the key listener, not through `editSelected()`, so the patch leaves that path alone — the view-only editor with its expand icon still opens, and long or multi-line values can still be read and selected there. But that is keyboard-only and nothing on screen says so, which is exactly your point: I removed the discoverable way in and left the one nobody finds. Double-click solves it cheaply — the table has a `mouseDown` listener but no `MouseDoubleClick` one today, so it is an addition, not a change anywhere else. Single click selects the row, double click opens the cell as a click does today. That also brings back select-the-text-and-copy for a single cell, so your second point largely goes with it. Happy to add a shortcut for *Copy cell value* too if you want one. I will push the double-click handler to this branch unless you would rather see it differently. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
