From 4802eb0514c2778e31e4c9b0525343e4dd2322d8 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Wed, 29 Jul 2015 19:37:51 +0300
Subject: [PATCH] minor fixed + found a bug...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 ui (gen4)/viewer.js | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/ui (gen4)/viewer.js b/ui (gen4)/viewer.js
index ba3c730d..0b49d575 100755
--- a/ui (gen4)/viewer.js	
+++ b/ui (gen4)/viewer.js	
@@ -695,11 +695,19 @@ actions.Actions({
 
 	cropRibbon: ['Crop current ribbon',
 		function(ribbon, flatten){
+			if(typeof(ribbon) == typeof(true)){
+				flatten = ribbon
+				ribbon = null
+			}
 			ribbon = ribbon || 'current'
 			this.crop(this.data.getImages(ribbon), flatten)
 		}],
 	cropRibbonAndAbove: ['Crop current and above ribbons',
 		function(ribbon, flatten){
+			if(typeof(ribbon) == typeof(true)){
+				flatten = ribbon
+				ribbon = null
+			}
 			ribbon = ribbon || this.data.getRibbon()
 
 			var data = this.data
@@ -1258,6 +1266,14 @@ actions.Actions(Client, {
 
 
 	crop: [ reloadAfter() ],
+	// XXX BUG: this does not align correctly if the current image is in
+	// 		a different ribbon relative to the top...
+	// 		to repeat:
+	// 			- in a three ribbon setup focus an image in the middle ribbon
+	// 			- .cropRibbonAndAbove(true) -> single ribbon
+	// 			- .uncrop()
+	// 		result:
+	// 			ribbon vertical alignment will not change.
 	uncrop: [ reloadAfter() ],
 	// XXX might be a good idea to do this in a new viewer in an overlay...
 	cropGroup: [ reloadAfter() ],