assets/js/romo/select_dropdown.js in romo-0.16.1 vs assets/js/romo/select_dropdown.js in romo-0.16.2

@@ -3,17 +3,17 @@
     return new RomoSelectDropdown(element, optionElemsParent);
   });
 }
 
 var RomoSelectDropdown = function(element, optionElemsParent) {
-  this.elem = $(element);
+  this.elem         = $(element);
   this.itemSelector = 'LI[data-romo-select-item="opt"]:not(.disabled)';
-  this.prevValue = undefined;
+  this.prevValue    = '';
 
-  var optsParent = (optionElemsParent || this.elem.find('.romo-select-dropdown-options-parent'));
+  var optsParent   = (optionElemsParent || this.elem.find('.romo-select-dropdown-options-parent'));
   this.optionElems = optsParent.children();
-  this.optionList = this._buildOptionList(this.optionElems);
+  this.optionList  = this._buildOptionList(this.optionElems);
 
   this.doInit();
   this.doBindDropdown();
 
   if (this.elem.attr('id') !== undefined) {
@@ -86,11 +86,11 @@
   this.romoDropdown.popupElem.on('mouseup',   $.proxy(this.onPopupMouseUp, this));
 }
 
 RomoSelectDropdown.prototype.doSelectHighlightedItem = function() {
   var prevValue = this.prevValue;
-  var newValue = this.romoDropdown.bodyElem.find('LI.romo-select-highlight').data('romo-select-option-value');
+  var newValue  = this.romoDropdown.bodyElem.find('LI.romo-select-highlight').data('romo-select-option-value');
 
   this.romoDropdown.doPopupClose();
   this.elem.trigger('selectDropdown:itemSelected', [newValue, prevValue, this]);
 
   if (newValue !== prevValue) {
@@ -230,16 +230,18 @@
   }, this));
   return list;
 }
 
 RomoSelectDropdown.prototype._buildOptionListItem = function(optionElem) {
-  var opt = $(optionElem);
-  var item = $('<li data-romo-select-item="opt"></li>');
+  var opt   = $(optionElem);
+  var item  = $('<li data-romo-select-item="opt"></li>');
+  var value = opt.attr('value') || '';
 
-  item.attr('data-romo-select-option-value', opt.attr('value'));
+  item.attr('data-romo-select-option-value', value);
   item.html(opt.text().trim() || '&nbsp;');
   if (opt.prop('selected')) {
     item.addClass('selected');
+    this.prevValue = value; // the last option marked selected is used
   }
   if (opt.attr('disabled') !== undefined) {
     item.addClass('disabled');
   }