fskorgen commented on PR #8530:
URL: https://github.com/apache/hop/pull/8530#issuecomment-5854226028

   Took your suggestion — double-click now opens the cell on a read-only grid, 
so the view-only editor
   with its expand icon is still one gesture away and long or multi-line values 
stay readable and
   selectable. Single click keeps selecting the row, which is what makes the 
context menu and the
   selection actions reachable.
   
   That also covers your second point: selecting the text in the editor and 
copying it works as before,
   alongside the *Copy cell value* menu item. Say the word if you still want a 
keyboard shortcut bound
   to that one.
   
   Rebased onto current main and squashed to a single commit. Ready for the new 
review.
   
   On the unlock toggle — I would rather bring that as its own issue than grow 
this one: it lands in
   `BaseDialog`, the metadata editors and `ProjectDialog` rather than in 
`TableView`, and it has an open
   question worth its own thread, namely who is allowed to unlock when Hop Web 
has no permission
   covering it. I will take you up on the offer to review it, just separately, 
so this fix can land.
   


-- 
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]

Reply via email to