From c0e678ace524f4356325b8eb05f847c42f1989c8 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Thu, 7 Jan 2016 03:58:18 +0300
Subject: [PATCH] some tuning and cleanup...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 ui (gen4)/features/external-editor.js |  2 ++
 ui (gen4)/index.html                  | 23 +++++++++++++++++++----
 2 files changed, 21 insertions(+), 4 deletions(-)

diff --git a/ui (gen4)/features/external-editor.js b/ui (gen4)/features/external-editor.js
index cfff05c2..083acd99 100755
--- a/ui (gen4)/features/external-editor.js	
+++ b/ui (gen4)/features/external-editor.js	
@@ -31,6 +31,7 @@ var browseWalk = require('lib/widget/browse-walk')
 // 		- image type to open
 // XXX add root button...
 // XXX disable the remove button on "System default"
+// XXX move the CSS out of index.html and into a correct CSS file...
 // XXX move this to a separate feature...
 
 var ExternalEditorActions = actions.Actions({
@@ -268,6 +269,7 @@ module.ExternalEditorUI = core.ImageGridFeatures.Feature({
 
 
 
+
 /**********************************************************************
 * vim:set ts=4 sw=4 :                                                */
 return module })
diff --git a/ui (gen4)/index.html b/ui (gen4)/index.html
index 07736d2f..f9bbdd9e 100755
--- a/ui (gen4)/index.html	
+++ b/ui (gen4)/index.html	
@@ -124,26 +124,41 @@ body {
 
 
 
-.browse-widget.editor-list .list div:first-child .text:after {
+/* External Editor List */
+.browse-widget.editor-list .list>div:first-child .text:after {
 	content: "(primary)";
 	margin-left: 5px;
 	opacity: 0.5;
 	font-style: italic;
 }
-.browse-widget.editor-list .list div:nth-child(2) .text:after {
+/* XXX this is ugly -- use a class... */
+.browse-widget.editor-list .list>div:first-child .button:nth-child(4) {
+	opacity: 0.1;
+}
+.browse-widget.editor-list .list>div:nth-child(2) .text:after {
 	content: "(secondary)";
 	margin-left: 5px;
 	opacity: 0.5;
 	font-style: italic;
 }
-.browse-widget.editor-list .list div:last-child .text {
+/* XXX this is ugly -- use a class... */
+.browse-widget.editor-list .list>div:nth-child(2) .button:nth-child(3) {
+	opacity: 0.1;
+}
+
+/* XXX STUB: use a button instead of a list element... */
+.browse-widget.editor-list .list>div:last-child {
+	border-top: solid 1px rgba(255,255,255, 0.2);
+}
+.browse-widget.editor-list .list>div:last-child .text {
 	font-style: italic;
 }
-.browse-widget.editor-list .list div:last-child .button {
+.browse-widget.editor-list .list>div:last-child .button {
 	display: none;
 }
 
 
+
 /* DEBUG stuff... */
 .container-center {
 	position: absolute;