x/merlin
Code Issues Proposed changes

Update DAL on 'change' event on input field

Browse Source 
This allows to remove 'Set' button and clear UI significantly.

Change-Id: I217c683a5409a7c3f7bd2e1d53b2c63279f09123
This commit is contained in:
Timur Sufiev 2014-10-01 19:49:10 +04:00
parent 548513b34a
commit 1262d528b6
1 changed files with 5 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
js/merlin.js
View File

@ -25,16 +25,14 @@ $(function() {
function drawBaseNode($label, item, type, converter) { function drawBaseNode($label, item, type, converter) {
var $item = $('<div></div>'), var $item = $('<div></div>'),
$input = $('<input>'), $input = $('<input>');
$set = $('<button>').text('Set');
converter = converter || function(x) { return x;}; converter = converter || function(x) { return x;};
$set.click(function() {
item.set(converter($input.val()));
});
$input.val(item.get()); $input.val(item.get());
$input.change(function() {
item.set(converter($input.val()))
});
$item.append($label); $item.append($label);
$item.append($input.attr('type', type)); $item.append($input.attr('type', type));
$item.append($set);
return $item; return $item;
} }
@ -50,14 +48,12 @@ $(function() {
function drawSelectNode($label, item) { function drawSelectNode($label, item) {
var $item = $('<div></div>'), var $item = $('<div></div>'),
$set = $('<button>').text('Set'),
$select = drawSelectElement(item.getEnumLabels(), item.getEnumValues(), item.get()); $select = drawSelectElement(item.getEnumLabels(), item.getEnumValues(), item.get());
$set.click(function() { $select.change(function() {
item.set($select.val()); item.set($select.val());
}); });
$item.append($label); $item.append($label);
$item.append($select); $item.append($select);
$item.append($set);
return $item; return $item;
} }