From 3aeaf6e36e5b25826116218dd4dcfde394efbd5c Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Mon, 24 Nov 2014 18:16:56 +0300
Subject: [PATCH] basic group discription...

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

diff --git a/ui (gen4)/data.js b/ui (gen4)/data.js
index 36542274..9bcac52c 100755
--- a/ui (gen4)/data.js	
+++ b/ui (gen4)/data.js	
@@ -1477,6 +1477,22 @@ var DataPrototype = {
 
 
 	/****************************************************** Groups ***/
+	// A group is a small set of images...
+	//
+	// All images in a group are in ONE ribbon but can be at any location
+	// in order.
+	//
+	// It can be in one of two states:
+	// 	- collapsed
+	// 		- group images are not loaded
+	// 		- group cover is loaded
+	// 	- expanded
+	// 		- group cover is not loaded
+	// 		- group images are loaded
+	//
+	// XXX group cover -- which image and should this question be asked
+	// 		on this level???
+	//
 	// XXX experimental...
 	// 		...not sure if storing groups in .groups here is the right 
 	// 		way to go...
diff --git a/ui (gen4)/viewer.js b/ui (gen4)/viewer.js
index f631e0bb..320f1d2d 100755
--- a/ui (gen4)/viewer.js	
+++ b/ui (gen4)/viewer.js	
@@ -1056,6 +1056,7 @@ actions.Actions(Client, {
 	groupMarked: [ reloadAfter() ],
 	expandGroup: [ reloadAfter() ],
 	collapseGroup: [ reloadAfter() ],
+	// XXX might be a good idea to do this in a new viewer in an overlay...
 	cropGroup: [ reloadAfter() ],
 
 	// XXX experimental: not sure if this is the right way to go...