Skip to content
Prev Previous commit
Next Next commit
Changed nargument names to valid identifers to avoid confusion
Argument names with whitespace like "jQuery object" are confusing,
because it might easily be mistaken for two arguments when reading the
signature. Argument names like "-index" are also confusing since they
look like an expression rather than a name. I have renamed the argument
names to be valid javascript identifiers, just like actual argument
names have to be. Also changed argument type "Object" and "PlainObject"
to "JQuery" where the prose documentation indicates that the type is a
jQuery object.
  • Loading branch information
esx committed Jan 19, 2014
commit d68f7aefa6dfa11aee3172c6f7cb4c92c5bb4a3e
2 changes: 1 addition & 1 deletion entries/add.xml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
</signature>
<signature>
<added>1.3.2</added>
<argument name="jQuery object" type="jQuery">
<argument name="jQueryObject" type="jQuery">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I'm missing something, but I'm not seeing anything effect of this change.

<desc>An existing jQuery object to add to the set of matched elements.</desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/closest.xml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
</signature>
<signature>
<added>1.6</added>
<argument name="jQuery object" type="jQuery">
<argument name="jQueryObject" type="jQuery">
<desc>A jQuery object to match elements against.</desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/eq-selector.xml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<signature>
<sample>:eq(-index)</sample>
<added>1.8</added>
<argument name="-index" type="Integer">
<argument name="indexFromEnd" type="Integer">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I'm missing something but I'm not seeing any effect of these name changes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as "jQuery object" to "selection", this is just to produce valid variable names in the signatures.

<desc>Zero-based index of the element to match, counting backwards from the last element. </desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/eq.xml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
</signature>
<signature>
<added>1.4</added>
<argument name="-index" type="Integer">
<argument name="indexFromEnd" type="Integer">
<desc>An integer indicating the position of the element, counting backwards from the last element in the set. </desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/filter.xml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
</signature>
<signature>
<added>1.4</added>
<argument name="jQuery object" type="Object">
<argument name="jQueryObject" type="Object">
<desc>An existing jQuery object to match the current set of elements against.</desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/find.xml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
</signature>
<signature>
<added>1.6</added>
<argument name="jQuery object" type="Object">
<argument name="jQuery" type="jQuery">
<desc>A jQuery object to match elements against.</desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/gt-selector.xml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<signature>
<sample>:gt(-index)</sample>
<added>1.8</added>
<argument name="-index" type="Integer">
<argument name="indexFromEnd" type="Integer">
<desc>Zero-based index, counting backwards from the last element. </desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/is.xml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
</signature>
<signature>
<added>1.6</added>
<argument name="jQuery object" type="Object">
<argument name="jQueryObject" type="Object">
<desc>An existing jQuery object to match the current set of elements against.</desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/jQuery.xml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
</signature>
<signature>
<added>1.0</added>
<argument name="jQuery object" type="PlainObject">
<argument name="jQueryObject" type="jQuery">
<desc>An existing jQuery object to clone.</desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/lt-selector.xml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<signature>
<sample>:lt(-index)</sample>
<added>1.8</added>
<argument name="-index" type="Integer">
<argument name="indexFromEnd" type="Integer">
<desc>Zero-based index, counting backwards from the last element. </desc>
</argument>
</signature>
Expand Down
2 changes: 1 addition & 1 deletion entries/not.xml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
</signature>
<signature>
<added>1.4</added>
<argument name="jQuery object" type="PlainObject">
<argument name="jQueryObject" type="jQuery">
<desc>An existing jQuery object to match the current set of elements against.</desc>
</argument>
</signature>
Expand Down