From 369578f2856ad417b1da29466f911457a2116c04 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Fri, 15 Jan 2016 05:37:06 +0300
Subject: [PATCH] basic work on display modes...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 ui (gen4)/features/exif.js     | 18 +++++++++---------
 ui (gen4)/lib/widget/browse.js | 14 +++++++++++---
 2 files changed, 20 insertions(+), 12 deletions(-)

diff --git a/ui (gen4)/features/exif.js b/ui (gen4)/features/exif.js
index 6f6c142b..b2d3f71b 100755
--- a/ui (gen4)/features/exif.js	
+++ b/ui (gen4)/features/exif.js	
@@ -219,6 +219,7 @@ var MetadataUIActions = actions.Actions({
 		],
 	},
 
+	// XXX should we replace 'mode' with nested set of metadata???
 	showMetadata: ['Image/Show metadata',
 		function(image, mode){
 			image = this.data.getImage(image)
@@ -242,13 +243,17 @@ var MetadataUIActions = actions.Actions({
 					.capitalize()
 
 				// skip metadata stuff in short mode...
-				if(mode == 'short' 
+				if(mode != 'full' 
 						&& field_order.indexOf(n) == -1){
-					return
+					if(mode == 'short'){
+						return
+
+					} else if(mode == 'disabled') {
+						n = '- ' + n
+					}
 				}
 
-				fields.push([ n + ': ',
-						metadata[k] ])
+				fields.push([ n + ': ', metadata[k] ])
 			})
 
 			// sort fields...
@@ -260,11 +265,6 @@ var MetadataUIActions = actions.Actions({
 				return a - b
 			})
 
-			// XXX replace field names with pretty names...
-			// XXX do two types of fields:
-			// 		- base
-			// 		- other (browse.Browse + sub-directory???)
-			
 			var o = overlay.Overlay(this.ribbons.viewer, 
 				browse.makeList(
 						null,
diff --git a/ui (gen4)/lib/widget/browse.js b/ui (gen4)/lib/widget/browse.js
index c889fbea..b491ee9e 100755
--- a/ui (gen4)/lib/widget/browse.js	
+++ b/ui (gen4)/lib/widget/browse.js	
@@ -2098,11 +2098,19 @@ ListPrototype.options = {
 				var n = k
 
 				// XXX make this support list args as well...
-				if(pattern && typeof(k) == typeof('str')){
-					var n = k.replace(pattern, '')
-					if(n != k){
+				if(pattern){
+					var t = typeof(n) == typeof('str') ? n : n[0]
+
+					var tt = t.replace(pattern, '')
+					if(t != tt){
 						disable = true
 
+						if(typeof(k) == typeof('str')){
+							n = tt
+						} else {
+							n[0] = tt
+						}
+
 						if(that.options.skipDisabledItems){
 							return
 						}