Refactor code.

This commit is contained in:
Lars Jung 2015-04-25 17:10:47 +02:00
parent d5221c0d17
commit 1a47a2aa3b
5 changed files with 33 additions and 40 deletions

View file

@ -44,14 +44,16 @@ modulejs.define('ext/contextmenu', ['_', '$', 'core/resource', 'core/settings'],
} else if (entry.type === 'l') { } else if (entry.type === 'l') {
$(templateLabel) $(templateLabel)
.find('.cm-text').text(entry.text).end() .appendTo($ul)
.appendTo($ul); .find('.cm-text')
.text(entry.text);
} else if (entry.type === 'e') { } else if (entry.type === 'e') {
$li = $(templateEntry) $li = $(templateEntry)
.appendTo($ul)
.data('cm-id', entry.id) .data('cm-id', entry.id)
.find('.cm-text').text(entry.text).end() .find('.cm-text')
.appendTo($ul); .text(entry.text);
if (entry.icon) { if (entry.icon) {
$li.find('.cm-icon img').attr('src', resource.icon(entry.icon)); $li.find('.cm-icon img').attr('src', resource.icon(entry.icon));

View file

@ -24,7 +24,7 @@ modulejs.define('ext/crumb', ['_', '$', 'core/event', 'core/location', 'core/res
.data('item', item); .data('item', item);
location.setLink($html, item); location.setLink($html, item);
$html.find('.label').text(item.label).end(); $html.find('.label').text(item.label);
if (item.isDomain() || item.isRoot()) { if (item.isDomain() || item.isRoot()) {
$html.find('.sep').width(0); $html.find('.sep').width(0);

View file

@ -153,9 +153,9 @@ modulejs.define('ext/preview', ['_', '$', 'core/resource', 'core/settings', 'cor
function setRawLink(href) { function setRawLink(href) {
if (href) { if (href) {
$('#pv-bar-raw').find('a').attr('href', href).end().show(); $('#pv-bar-raw').show().find('a').attr('href', href);
} else { } else {
$('#pv-bar-raw').find('a').attr('href', '#').end().hide(); $('#pv-bar-raw').hide().find('a').attr('href', '#');
} }
} }

View file

@ -125,31 +125,28 @@ modulejs.define('ext/sort', ['_', '$', 'core/event', 'core/resource', 'core/sett
return; return;
} }
$('#items li.header') var $header = $('#items li.header');
.find('a.label') $header.find('a.label')
.append(template) .append(template)
.click(function (ev) { .click(function (ev) {
sortItems(0, $(this).hasClass('ascending')); sortItems(0, $(this).hasClass('ascending'));
ev.preventDefault(); ev.preventDefault();
}) });
.end()
.find('a.date') $header.find('a.date')
.prepend(template) .prepend(template)
.click(function (ev) { .click(function (ev) {
sortItems(1, $(this).hasClass('ascending')); sortItems(1, $(this).hasClass('ascending'));
ev.preventDefault(); ev.preventDefault();
}) });
.end()
.find('a.size') $header.find('a.size')
.prepend(template) .prepend(template)
.click(function (ev) { .click(function (ev) {
sortItems(2, $(this).hasClass('ascending')); sortItems(2, $(this).hasClass('ascending'));
ev.preventDefault(); ev.preventDefault();
}) });
.end();
event.sub('location.changed', onContentChanged); event.sub('location.changed', onContentChanged);
event.sub('location.refreshed', onContentChanged); event.sub('location.refreshed', onContentChanged);

View file

@ -23,18 +23,12 @@ modulejs.define('main/info', ['$', 'config'], function ($, config) {
function addTest(label, info, passed, result) { function addTest(label, info, passed, result) {
$(testTemp) var $test = $(testTemp).appendTo('#tests');
.find('.label') $test.find('.label').text(label);
.text(label) $test.find('.result')
.end()
.find('.result')
.addClass(passed ? 'passed' : 'failed') .addClass(passed ? 'passed' : 'failed')
.text(result ? result : (passed ? 'yes' : 'no')) .text(result ? result : (passed ? 'yes' : 'no'));
.end() $test.find('.info').html(info);
.find('.info')
.html(info)
.end()
.appendTo('#tests');
} }
function addTests() { function addTests() {