#148 new
nickg

replace __isIdDeclaration__ with Attr.isId

Reported by nickg | April 7th, 2010 @ 04:58 PM

ok this is not a really major problem, but...

in dom/domimplementation.js we have the function:

function __isIdDeclaration__(attributeName) {
  // test if attributeName is 'id' (case insensitive)                           
  return attributeName?(attributeName.toLowerCase() == 'id'):false;
}

ala DOM Core Level 3:

getElementById introduced in DOM Level 2

Returns the Element that has an ID attribute with the given value. If no such element exists, this returns null. If more than one element has an ID attribute with that value, what is returned is undefined. 

The DOM implementation is expected to use the attribute Attr.isId to determine if an attribute is of type ID.

Note: Attributes with the name "ID" or "id" are not of type ID unless so defined.

So seems like we can move the custom function into dom/attr.js itself.

yeah it's minor cleanup task

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

a javascript browser environment

People watching this ticket

Tags

Pages