switch deleteButtonWithCheck to use a red circle minus icon consistent with the new green plus icon

Author: Tim Berners-Lee

lib/widgets.js

@@ -178,13 +178,20 @@ widgetModule.setImage = function (element, x) {
 //   @@ Supress check if command key held down?
 //
 widgetModule.deleteButtonWithCheck = function (dom, container, noun, deleteFunction) {
-  var delButton = dom.createElement('button')
-  container.appendChild(delButton)
-  delButton.textContent = '-'
+  var minusIconURI = UI.icons.iconBase + 'noun_2188_red.svg' // white minus in red #cc0000 circle
+
+  //var delButton = dom.createElement('button')
+
+  var img = dom.createElement('img')
+  img.setAttribute('src', minusIconURI) //  plus sign
+  img.setAttribute('style', 'margin: 0.2em; width: 1em; height:1em')
+  img.title = 'Remove this ' + noun
+  var delButton = img
 
+  container.appendChild(delButton)
   container.setAttribute('class', 'hoverControl') // See tabbedtab.css (sigh global CSS)
   delButton.setAttribute('class', 'hoverControlHide')
-  delButton.setAttribute('style', 'color: red; margin-right: 0.3em; foat:right; text-align:right')
+  // delButton.setAttribute('style', 'color: red; margin-right: 0.3em; foat:right; text-align:right')
   delButton.addEventListener('click', function (e) {
     container.removeChild(delButton) // Ask -- are you sure?
     var cancelButton = dom.createElement('button')
@@ -404,6 +411,7 @@ widgetModule.field[UI.ns.ui('Multiple').uri] = function (
   dom, container, already, subject, form, store, callback) {
   // var plusIcon = UI.icons.originalIconBase + 'tango/22-list-add.png' // blue plus
   var plusIconURI = UI.icons.iconBase + 'noun_19460_green.svg' // white plus in green circle
+
   var kb = UI.store
   kb.updater = kb.updater || new $rdf.UpdateManager(kb)
   var box = dom.createElement('table')
@@ -483,8 +491,8 @@ widgetModule.field[UI.ns.ui('Multiple').uri] = function (
     UI.utils.label(property)
 
   values.map(function (obj) { addItem(null, obj) })
-
-  while (values.length < min) {
+  var extra = min - values.length
+  for (var j=0; j < extra; j++) {
     console.log("Adding extra: min " + min )
     addItem() // Add blanks if less than minimum
   }
@@ -541,11 +549,11 @@ widgetModule.fieldParams[UI.ns.ui('FloatField').uri].pattern =
   widgetModule.fieldParams[UI.ns.ui('NamedNodeURIField').uri] = { namedNode: true}
 widgetModule.fieldParams[UI.ns.ui('TextField').uri] = { }
 
-widgetModule.fieldParams[UI.ns.ui('PhoneField').uri] = { 'size': 12, 'uriPrefix': 'tel:' }
+widgetModule.fieldParams[UI.ns.ui('PhoneField').uri] = { 'size': 20, 'uriPrefix': 'tel:' }
 widgetModule.fieldParams[UI.ns.ui('PhoneField').uri].pattern =
   /^\s*\+?[ 0-9-]+[0-9]\s*$/
 
-widgetModule.fieldParams[UI.ns.ui('EmailField').uri] = { 'size': 20, 'uriPrefix': 'mailto:' }
+widgetModule.fieldParams[UI.ns.ui('EmailField').uri] = { 'size': 30, 'uriPrefix': 'mailto:' }
 widgetModule.fieldParams[UI.ns.ui('EmailField').uri].pattern =
   /^\s*.*@.*\..*\s*$/ // @@ Get the right regexp here