Women in Technology

Hear us Roar

Article:		Cooking with JavaScript & DHTML
Subject:		a better mousetrap
Date:		2006-06-28 07:17:41
From:		jason2584

I've messed with Danny's script pretty extensively, and I ended up solving several of the problems mentioned here before I ran into the issue of the change event not firing for the select element. I'm still looking for the fix to that one, but I thought I'd post my code that doesn't have the num-pad problem or the problem where typing beyond a find starts the search over instead of staying on the first match. It's also shorter than the original, but I removed the comments, so you'll have to do your own dirty work to make changes to it. I hope it helps someone. `var taInfo = { last:0, charsTyped:'', delay:1000, timeout:null, reset:function() { this.last=0; this.charsTyped=''; } };` function typeAhead() { if(window.event && !window.event.ctrlKey && window.event.keyCode > 19) { var now = new Date(); if(taInfo.charsTyped == '' \|\| now - taInfo.last < taInfo.delay) { var ddl = window.event.srcElement; clearTimeout(taInfo.timeout); taInfo.last = now; taInfo.timeout = setTimeout('taInfo.reset()', taInfo.delay); taInfo.charsTyped += String.fromCharCode(window.event.keyCode).toLowerCase(); for(var i=0; i < ddl.options.length-1; i++) if(ddl.options[i].text.toLowerCase().indexOf(taInfo.charsTyped) == 0) { ddl.selectedIndex = i; break; } } window.event.returnValue = false; return true; } return true; }

Showing messages 1 through 1 of 1.

oops
2006-06-28 07:26:06 jason2584 [View]

The terminating expression on the for loop in the code I posted was incorrect. It should run for i < ddl.options.length, not i < ddl.options.length-1. My bad.