MakeItSans

By mva.led — Last update Apr 25, 2008 — Installed 278 times.

Little bug

in
Subscribe to Little bug 2 posts, 2 voices



Apr 9, 2009 6:36am
Peter Farmer Scriptwright

A JS checker noted a couple of issues with the code: the 'for' and 'if' statements in Lines 73-75 don't have opening and closing brackets.

Suggest changing these to

for(var i = 0; i < targets.length; i++) {
if (shouldSans(targets[i].getAttribute('FACE'))) {
targets[i].setAttribute('FACE', 'sans-serif');
}
}
 
Sep 2, 2009 7:38pm
mva.led Script's Author
MozillaX11

Thanks, for the report. I haven't had time to look at this issues. I see you have implemented another script with similar purposes. I've just peek at your code and it seems a *lot* simpler than mine. So I probably halt using mine script and use yours.

Best regards,
Manuel.
Cross
Presentational HTML allowed.
Use <code> for inline code and <pre> for code blocks. Use &lt; and &gt; for literal < and >.
We help break paragraphs and link your links.
or cancel