From 6da746012dbf9f33e3595c231dc0e67cc9e06574 Mon Sep 17 00:00:00 2001 From: "Alex A. Naanou" Date: Tue, 26 Jan 2016 07:43:22 +0300 Subject: [PATCH] added a couple of notes... Signed-off-by: Alex A. Naanou --- ui (gen4)/features/metadata.js | 1 + ui (gen4)/lib/widget/browse.js | 3 +++ 2 files changed, 4 insertions(+) diff --git a/ui (gen4)/features/metadata.js b/ui (gen4)/features/metadata.js index e9d76b61..d430f157 100755 --- a/ui (gen4)/features/metadata.js +++ b/ui (gen4)/features/metadata.js @@ -391,6 +391,7 @@ var MetadataUIActions = actions.Actions({ // add separator to base... fields.length > 0 && base.push('---') + // XXX might be a good idea to directly bind ctrl-c to copy value... var o = overlay.Overlay(this.ribbons.viewer, browse.makeList( null, diff --git a/ui (gen4)/lib/widget/browse.js b/ui (gen4)/lib/widget/browse.js index 5c0226ee..d7dcd724 100755 --- a/ui (gen4)/lib/widget/browse.js +++ b/ui (gen4)/lib/widget/browse.js @@ -183,6 +183,9 @@ var BrowserClassPrototype = { // XXX Q: should we make a base list dialog and build this on that or // simplify this to implement a list (removing the path and disabling // traversal)?? +// XXX might be a good idea to add a ctrl-c/copy handler... +// ...copy path by default but overloadable with something like +// .getCopyValue() which would return .strPath by default... var BrowserPrototype = { dom: null,