From 3e5c52b676ea4600eb547cab2a648a14135fb9ca Mon Sep 17 00:00:00 2001 From: "Alex A. Naanou" Date: Sun, 29 Jan 2017 06:20:58 +0300 Subject: [PATCH] added space triggering of items in browse lists... Signed-off-by: Alex A. Naanou --- ui (gen4)/features/keyboard.js | 5 ++--- ui (gen4)/lib/widget/browse.js | 7 +++++-- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/ui (gen4)/features/keyboard.js b/ui (gen4)/features/keyboard.js index 6f3011a9..f19c1c8c 100755 --- a/ui (gen4)/features/keyboard.js +++ b/ui (gen4)/features/keyboard.js @@ -394,8 +394,6 @@ module.GLOBAL_KEYBOARD = { /*********************************************************************/ // XXX add loading/storing of kb bindings... -// XXX experimenting with new style of doc strings, usable from the -// system... (for details see: core.doc) // XXX need a clean deep copy to restore... var KeyboardActions = actions.Actions({ @@ -1572,7 +1570,7 @@ var KeyboardUIActions = actions.Actions({ })], - // XXX move to gen2 + /*/ XXX move to gen2 // XXX need to pre-process the docs... // - remove the path component... // - insert the action name where not doc present... @@ -1590,6 +1588,7 @@ var KeyboardUIActions = actions.Actions({ background: 'white', focusable: true, })], + //*/ }) diff --git a/ui (gen4)/lib/widget/browse.js b/ui (gen4)/lib/widget/browse.js index 63ec5204..ba1ba204 100755 --- a/ui (gen4)/lib/widget/browse.js +++ b/ui (gen4)/lib/widget/browse.js @@ -1237,7 +1237,9 @@ var BrowserPrototype = { End: 'navigate!: "last"', Enter: 'action', + Space: 'Enter', //O: 'action', + Esc: 'close: "reject"', '/': 'startFilter!', @@ -1676,8 +1678,9 @@ var BrowserPrototype = { // ...this will also need to indicate a way to split the path // and when to 'pop' the sub browser... // XXX should we use the button tag for item buttons??? - // ...tried it, seems that with buttons we get some css overridden - // by Chrome, needs more work than just replacing a tag... + // ...basically for this to work we need to either reset or override + // user-agent-stylesheet... + // to override just set most of the affected options to inherit... update: function(path, list){ path = path || this.path var browser = this.dom