Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
This commit is contained in:
Alex A. Naanou 2014-10-22 05:58:31 +04:00
parent 69dcca5c37
commit 0b32958599

View File

@ -620,11 +620,12 @@ actions.Actions(Client, {
// The main question here is: // The main question here is:
// should we split out aligning to a feature? // should we split out aligning to a feature?
// The differences/trade-off's in this version: // 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 in one place
// + all the logic in one place // + all the logic in one place
// + usable as-is without any extra "features" // + usable as-is without any extra "features"
// - not customizable // - not customizable without rewriting...
// - might be too monolithic (god object?) // - might be too monolithic (god object?)
focusImage: [ focusImage: [
function(target, list){ function(target, list){