One comment is that you're adding a memory overhead for each event listener you're attaching. For a half-dozen rollovers, this isn't a problem, but you could potentially be attaching several dozen of them.
In such a case, you'd want to attach one event listener on a parent element and check for individual cases by determining the target of the button click. So, for instance, something like the following:
var over = function(eventObject) {
// get around IE's quirky DOM handling
var target = (eventObject.target) ? eventObject.target : eventObject.srcElement
if (YAHOO.util.Dom.hasClass(target, 'btn')) {
// it means we have a menu button
YAHOO.util.Dom.addClass(target, 'btn-over');
}
}
YAHOO.util.Event.on('menu', 'mouseover', over);