From 0b32958599657087641db6b1c09910310fd8c0cb Mon Sep 17 00:00:00 2001 From: "Alex A. Naanou" Date: Wed, 22 Oct 2014 05:58:31 +0400 Subject: [PATCH] ... Signed-off-by: Alex A. Naanou --- ui (gen4)/viewer.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ui (gen4)/viewer.js b/ui (gen4)/viewer.js index 0ae79d65..b395bd24 100755 --- a/ui (gen4)/viewer.js +++ b/ui (gen4)/viewer.js @@ -620,11 +620,12 @@ actions.Actions(Client, { // The main question here is: // should we split out aligning to a feature? // The differences/trade-off's in this version: - // + less code (not by much) + // + less code in total (not by much) + // 34 action-only vs. 39 total (25 action + 14 feature) // + all in one place // + all the logic in one place // + usable as-is without any extra "features" - // - not customizable + // - not customizable without rewriting... // - might be too monolithic (god object?) focusImage: [ function(target, list){