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

   Separate thought, not for this PR: where read-only is a *mode* rather than a 
property of the data —
   metadata editors opened read-only through `BaseDialog.setControlsReadOnly`, 
or `ProjectDialog`
   which already does `setReadonly(!editable)` — an unlock toggle in the dialog 
header would fit better
   than either click behaviour. The viewer grids here are the opposite case: 
their rows are a snapshot
   that is never read back, so there is nothing to unlock. Happy to write it up 
separately if it
   appeals.


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