The following is a couple bugfixes in Selector (just some unqualified
names that have been lurking)
There's also a few times in Selector where MochiKit.Format.trim could
be used, but isn't. It's not too big a deal, but it seems like if it's
already in MochiKit, it should be used in MochiKit.
Also a minor addition to DOM renames since IE blows and doesn't
recognize cellspacing.
Hope this is useful to you guys
- Aaron Faanes
===================================================================
--- DOM.js (revision 1262)
+++ DOM.js (working copy)
@@ -1139,6 +1139,8 @@
"for": "htmlFor",
"readonly": "readOnly",
"colspan": "colSpan",
+ "cellspacing":"cellSpacing",
+ "cellpadding":"cellPadding",
"bgcolor": "bgColor"
};
} else {
Index: Selector.js
===================================================================
--- Selector.js (revision 1262)
+++ Selector.js (working copy)
@@ -292,7 +292,7 @@
/** @id MochiKit.Selector.Selector.prototype.isUIElement */
isUIElement: function (element) {
- return findValue(['input', 'button', 'select', 'option',
'textarea', 'object'],
+ return MochiKit.Iter.findValue(['input', 'button', 'select',
'option', 'textarea', 'object'],
element.tagName.toLowerCase()) > -1;
},
@@ -341,7 +341,7 @@
}
if (axis == "") {
- scope = (scope ||
currentDocument()).getElementsByTagName(this.params.tagName || '*');
+ scope = (scope ||
MochiKit.DOM.currentDocument()).getElementsByTagName(this.params.tagName
|| '*');
} else if (axis == ">") {
if (!scope) {
throw "> combinator not allowed without preceeding
expression";
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"MochiKit" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/mochikit?hl=en
-~----------~----~----~----~------~----~------~--~---
