style: just use line-comments consistently - jscancer - Javascript crap (relati… | |
git clone git://git.codemadness.org/jscancer | |
Log | |
Files | |
Refs | |
README | |
LICENSE | |
--- | |
commit 9e4e376cbeea4ec4a09735e0b16c698d875954f0 | |
parent 556bb20940c177102090c1cc0e0c2d5142e3ced1 | |
Author: Hiltjo Posthuma <[email protected]> | |
Date: Thu, 2 Jun 2016 19:05:21 +0200 | |
style: just use line-comments consistently | |
Diffstat: | |
M datalist/datalist.js | 2 +- | |
M datatable/datatable.js | 37 +++++++++++++++--------------… | |
M datatable/example-ajax.html | 2 +- | |
M datepicker/datepicker.js | 19 +++++++++---------- | |
M tooltipxhr/tooltipxhr.js | 2 +- | |
5 files changed, 30 insertions(+), 32 deletions(-) | |
--- | |
diff --git a/datalist/datalist.js b/datalist/datalist.js | |
@@ -42,7 +42,7 @@ function datalist_init(input) { | |
if (f) | |
fc++; | |
} | |
- /* all tokens (separated by space) must match. */ | |
+ // all tokens (separated by space) must match. | |
if (fc == tok.length) | |
matches.push(data[i]); | |
} | |
diff --git a/datatable/datatable.js b/datatable/datatable.js | |
@@ -31,9 +31,9 @@ function datatable_init(el) { | |
ths: ths, | |
tbody: tbody[0], | |
cols: cols, | |
- sort: [], /* sort options: [colidx, order (ASC = 0, DESC = 1)… | |
+ sort: [], // sort options: [colidx, order (ASC = 0, DESC = 1)… | |
lazyscroll: ["1", "true"].indexOf(el.getAttribute("data-lazysc… | |
- search: "" /* previous search text. */ | |
+ search: "" // previous search text. | |
}; | |
d.data_raw = d.data = datatable_data_parse(d); | |
@@ -57,7 +57,7 @@ function datatable_init(el) { | |
var newths = []; | |
for (var i = 0; i < ths.length; i++) | |
newths.push(tr.appendChild(ths[i].cloneNode(true))); | |
- d.ths = newths; /* set new columns (for sorting etc).. */ | |
+ d.ths = newths; // set new columns (for sorting etc).. | |
var elthead = document.createElement("thead"); | |
elthead.appendChild(tr); | |
@@ -101,7 +101,7 @@ function datatable_init(el) { | |
startfiller.style.height = (start * rowheight) + "px"; | |
endfiller.style.height = ((nrows - end - 1) * rowheigh… | |
- /* remove nodes but keep first startfiller and endfill… | |
+ // remove nodes but keep first startfiller and endfill… | |
for (var c = bodytbody.childNodes; c.length > 2; ) | |
bodytbody.removeChild(startfiller.nextSibling); | |
@@ -111,11 +111,11 @@ function datatable_init(el) { | |
var curscrollleft, curscrolltop, verticalscrolltimer; | |
bodyel.addEventListener("scroll", function() { | |
- /* handle left / right scroll */ | |
+ // handle left / right scroll. | |
var scrolleft = bodyel.scrollLeft; | |
if (curscrollleft !== scrolleft) | |
headersel.scrollLeft = curscrollleft = scrolle… | |
- /* handle up/down scroll */ | |
+ // handle up/down scroll. | |
var scrolltop = bodyel.scrollTop; | |
if (curscrolltop !== scrolltop) { | |
clearTimeout(verticalscrolltimer); | |
@@ -135,11 +135,11 @@ function datatable_init(el) { | |
d.tbody = tbody; | |
}; | |
} | |
- /* setup click event handlers for sorting. */ | |
+ // setup click event handlers for sorting. | |
for (var i = 0; i < d.ths.length; i++) | |
d.cols[i].sortable && d.ths[i].addEventListener("click", funct… | |
return function(e) { | |
- /* shift-click for multi-select modifier. */ | |
+ // shift-click for multi-select modifier. | |
datatable_sort_column_toggle(d, idx, e.shiftKe… | |
d.data = datatable_sort(d, d.data); | |
d.display(d.data); | |
@@ -173,7 +173,7 @@ function datatable_sort_column_set(d, idx, order, multi) { | |
d.ths[d.sort[i][0]].className += " sort-" + (d.sort[i][1] ? "d… | |
} | |
-/* toggle sort or use default order: ASC. */ | |
+// toggle sort or use default order: ASC. | |
function datatable_sort_column_toggle(d, idx, multi) { | |
var c = datatable_sort_column_get(d, idx); | |
datatable_sort_column_set(d, idx, c == -1 || d.sort[c][1] ? 0 : 1, mul… | |
@@ -181,14 +181,14 @@ function datatable_sort_column_toggle(d, idx, multi) { | |
function datatable_data_parse(d) { | |
var data = [], trs = d.tbody.children; | |
- /* NOTE: assumes each tr has only "<td>" childnodes. */ | |
+ // NOTE: assumes each tr has only "<td>" childnodes. | |
for (var i = 0; i < trs.length; i++) { | |
var values = [], fv = []; | |
for (var j = 0, trc = trs[i].children; j < trc.length; j++) { | |
var td = trc[j], v = td.getAttribute("data-value"); | |
- /* prefer data-value attribute, else use cell contents, | |
- * also set preprocess values to filter on cell content | |
- * and data-value (case-insensitive). */ | |
+ // prefer data-value attribute, else use cell contents, | |
+ // also set preprocess values to filter on cell content | |
+ // and data-value (case-insensitive). | |
var s = (td.textContent || td.innerText).toLowerCase(); | |
if (typeof(v) != "undefined" && v !== null) { | |
fv.push([ v.toLowerCase(), s ]); | |
@@ -208,7 +208,7 @@ function datatable_data_parse(d) { | |
} | |
function datatable_sort(d, data) { | |
- /* setup sort functions once (in order for multi-select). */ | |
+ // setup sort functions once (in order for multi-select). | |
var sortfns = d.sort.map(function(s) { | |
return (function(c, o, fn) { | |
if (o) | |
@@ -243,7 +243,7 @@ function datatable_filter(d, data, s) { | |
if (f) | |
fc++; | |
} | |
- /* all tokens (separated by space) must match. */ | |
+ // all tokens (separated by space) must match. | |
if (fc == tok.length) | |
ret.push(data[i]); | |
} | |
@@ -270,7 +270,7 @@ function datatable_filter_delayed(d, fn, e) { | |
clearTimeout(d.filter_timer); | |
d.filter_timer = setTimeout(function() { | |
fn(e); | |
- }, 150); /* filter delay in ms. */ | |
+ }, 150); // filter delay in ms. | |
} | |
function datatable_autoload() { | |
@@ -282,7 +282,7 @@ function datatable_autoload() { | |
if ((d = datatable_init(dl[i])) === null) | |
continue; | |
var input = dl[i].parentNode.getElementsByClassName("filter-te… | |
- /* delayed filtering. */ | |
+ // delayed filtering. | |
for (var j = 0; j < input.length; j++) { | |
input[j].addEventListener("input", (function(d, el) { | |
return function(e) { | |
@@ -295,4 +295,4 @@ function datatable_autoload() { | |
ds.push(d); | |
} | |
return ds; | |
-} | |
-\ No newline at end of file | |
+} | |
diff --git a/datatable/example-ajax.html b/datatable/example-ajax.html | |
@@ -30,7 +30,7 @@ | |
<script type="text/javascript"> | |
function datatable_load_data(d, data) { | |
- /* NOTE: assumes each tr has only "<td>" childnodes. */ | |
+ // NOTE: assumes each tr has only "<td>" childnodes. | |
for (var i = 0; i < data.length; i++) { | |
var values = [], fv = []; | |
var tr = document.createElement("tr"); | |
diff --git a/datepicker/datepicker.js b/datepicker/datepicker.js | |
@@ -38,7 +38,7 @@ function datepicker_init(input) { | |
td = document.createElement("td"); | |
td.colSpan = 5; | |
td.className = "name"; | |
- var header = td; /* set header */ | |
+ var header = td; // set header. | |
tr.appendChild(td); | |
td = document.createElement("td"); | |
td.className = "next"; | |
@@ -83,14 +83,14 @@ function datepicker_init(input) { | |
left += c.offsetLeft; | |
} | |
table.style.left = String(left) + "px"; | |
- /* scroll if outside window. */ | |
+ // scroll if outside window. | |
input.scrollIntoView(); | |
}; | |
var datepicker_nextmon = function(n) { | |
curdate = new Date(curdate.getFullYear(), curdate.getMonth() +… | |
}; | |
var datepicker_render = function() { | |
- datepicker_nextmon(0); /* fix date */ | |
+ datepicker_nextmon(0); // fix date. | |
var y = curdate.getFullYear(), m = curdate.getMonth(); | |
header.innerHTML = datepicker_mons[m] + " " + String(y); | |
@@ -124,7 +124,7 @@ function datepicker_init(input) { | |
td.setAttribute("data-value", String(y) + "-" + pad0(m… | |
td.appendChild(document.createTextNode(String(i))); | |
- /* check if valid date (enabled or disabled). */ | |
+ // check if valid date (enabled or disabled). | |
var classes = []; | |
var d = parsedateutc(td.getAttribute("data-value")) ||… | |
if ((!isNaN(mindate) && d < mindate) || | |
@@ -132,7 +132,7 @@ function datepicker_init(input) { | |
classes.push("d"); | |
} else { | |
classes.push("v"); | |
- /* NOTE: onmousedown is handled before input.b… | |
+ // NOTE: onmousedown is handled before input.b… | |
td.addEventListener("mousedown", function(e) { | |
input.value = this.getAttribute("data-… | |
curdate = seldate = datepicker_datepar… | |
@@ -140,10 +140,10 @@ function datepicker_init(input) { | |
return !!e.stopPropagation(); | |
}, false); | |
} | |
- /* selected date? */ | |
+ // selected date? | |
if (!isNaN(sel) && sel >= d && sel < d + 86400000) | |
classes.push("sel"); | |
- /* is date today? */ | |
+ // is date today? | |
if (now.getFullYear() == y && now.getMonth() == m && n… | |
classes.push("today"); | |
td.className = classes.join(" "); | |
@@ -177,8 +177,7 @@ function datepicker_init(input) { | |
datepicker_render(); | |
datepicker_show(); | |
} else if (!inparent(table, e.target)) { | |
- /* is in parent? ignore */ | |
- /* format date onblur. */ | |
+ // is in parent? ignore, format date onblur. | |
var d = datepicker_dateparse(this.value); | |
if (!isNaN(d)) { | |
curdate = seldate = d; | |
@@ -198,7 +197,7 @@ function datepicker_init(input) { | |
input.parentNode.insertBefore(table, input.nextSibling); | |
} | |
-/* has native HTML5 date input type support? type is "text" if it isn't. */ | |
+// has native HTML5 date input type support? type is "text" if it isn't. | |
if (!(function() { var input = document.createElement("input"); try { input.ty… | |
var els = document.getElementsByClassName && document.getElementsByCla… | |
for (var i = 0; i < els.length; i++) | |
diff --git a/tooltipxhr/tooltipxhr.js b/tooltipxhr/tooltipxhr.js | |
@@ -10,7 +10,7 @@ | |
if (y + el.offsetHeight + 32 > window.innerHeight) | |
el.style.top = String(y - el.offsetHeight - 18) + "px"; | |
else | |
- el.style.top = String(y + 18) + "px"; /* + space for c… | |
+ el.style.top = String(y + 18) + "px"; // + space for c… | |
// NOTE, this is a mess: document.body.clientWidth excludes sc… | |
// window.innerWidth is the window including scrollbar wi… | |
// window.outerWidth is the window including scrollbar wi… |