added isFirst method

fixed isLast method

Author: maxazan

CHANGELOG.txt

@@ -1,4 +1,8 @@
-== 0.1.6
+== 0.1.7 - 15 November 2013
+* Fix isLast method. https://github.com/maxazan/jquery-treegrid/issues/4
+* Add isFirst method
+
+== 0.1.6 - 14 November 2013
 * Add isNode method
 * Add getAllNodes method
 * Fix isLast method. Throw error if method is not node of tree.

index.html

@@ -279,7 +279,7 @@ <h3>Public methods</h3>
                 </tr>          
                 <tr>
                     <td>isLast</td>
-                    <td>Is node a last element of its parent</td>
+                    <td>Return true if node is last in branch</td>
                     <td>
                         <pre><code class="javascript">// hide all last elements
 $('.tree').find('tr').each(function(){
@@ -289,6 +289,18 @@ <h3>Public methods</h3>
 });</code></pre>
                     </td>
                 </tr>
+                <tr>
+                    <td>isFirst</td>
+                    <td>Return true if node is first in branch</td>
+                    <td>
+                        <pre><code class="javascript">// hide all last elements
+$('.tree').find('tr').each(function(){
+  if ($(this).treegrid('isFirst')){
+	$(this).hide();
+  }
+});</code></pre>
+                    </td>
+                </tr>                
                 <tr>
                     <td>isExpanded</td>
                     <td>Is node expanded</td>

js/jquery.treegrid.js

@@ -1,5 +1,5 @@
 /*
- * jQuery treegrid Plugin 0.1.5
+ * jQuery treegrid Plugin 0.1.7
  * https://github.com/maxazan/jquery-treegrid
  * 
  * Copyright 2013, Pomazan Max
@@ -257,7 +257,7 @@
       }
       return $(this).treegrid('getParentNode').treegrid('getDepth') + 1;
     },
-    /*
+    /**
      * Method return true if node is root
      * 
      * @returns {Boolean}
@@ -279,11 +279,39 @@
      * @returns {Boolean}
      */
     isLast: function() {
-      if ($(this).next() && $(this).next().treegrid('isNode') && $(this).next().treegrid('getParentNodeId') === $(this).treegrid('getParentNodeId')) {
-        return false;
-      } else {
-        return true;
+      if ($(this).treegrid('isNode')) {
+        var parentNode = $(this).treegrid('getParentNode');
+        if (parentNode === null) {
+          if ($(this).treegrid('getNodeId') === $(this).treegrid('getRootNodes').last().treegrid('getNodeId')) {
+            return true;
+          }
+        } else {
+          if ($(this).treegrid('getNodeId') === parentNode.treegrid('getChildNodes').last().treegrid('getNodeId')) {
+            return true;
+          }
+        }
+      }
+      return false;
+    },
+    /**
+     * Method return true if node first in branch
+     * 
+     * @returns {Boolean}
+     */
+    isFirst: function() {
+      if ($(this).treegrid('isNode')) {
+        var parentNode = $(this).treegrid('getParentNode');
+        if (parentNode === null) {
+          if ($(this).treegrid('getNodeId') === $(this).treegrid('getRootNodes').first().treegrid('getNodeId')) {
+            return true;
+          }
+        } else {
+          if ($(this).treegrid('getNodeId') === parentNode.treegrid('getChildNodes').first().treegrid('getNodeId')) {
+            return true;
+          }
+        }
       }
+      return false;
     },
     /**
      * Return true if node expanded

test.html

@@ -45,6 +45,10 @@
       <tr class='treegrid-8 treegrid-parent-7' id="node-1-4-1">
         <td>1.4.1</td><td>Simple text of ...</td>
       </tr>
+      <tr class='treegrid-12 treegrid-parent-1' id="node-1-5">
+        <td>1.5</td><td>Simple text of ...</td>
+      </tr>
+	  
       <tr class="treegrid-11" id="node-2">
         <td>2</td><td>Simple text of ...</td>
       </tr>

tests/tests.js

@@ -35,7 +35,7 @@ test("getParentNode()", function() {
 });
 
 test("getChildNodes()", function() {
-  equal($('#node-1').treegrid('getChildNodes').length, 4, "Return 4");
+  equal($('#node-1').treegrid('getChildNodes').length, 5, "Return 5");
   equal($('#node-1-1-2-1').treegrid('getChildNodes').length, [], "Return []");
 });
 
@@ -50,10 +50,27 @@ test("isLeaf()", function() {
 });
 
 test("isLast()", function() {
+  //Test root nodes
+  ok($('#node-1').treegrid('isLast') === false, "Not Last!");
+  ok($('#node-2').treegrid('isLast') === true, "Last!");
+  //Test nodes with branch
+  ok($('#node-1-4').treegrid('isLast') === false, "Not Last test!");
+  ok($('#node-1-5').treegrid('isLast') === true, "Last test!");
   ok($('#node-1-1-2-1').treegrid('isLast') === true, "Last test!");
   ok($('#node-1-2').treegrid('isLast') === false, "Not Last test!");
 });
 
+test("isFirst()", function() {
+  //Test root nodes
+  ok($('#node-1').treegrid('isFirst') === true, "First!");
+  ok($('#node-2').treegrid('isFirst') === false, "Not first!");
+  //Test nodes with branch
+  ok($('#node-1-1').treegrid('isFirst') === true, "First!");
+  ok($('#node-1-4').treegrid('isFirst') === false, "Not first!");
+  ok($('#node-1-1-2-1').treegrid('isFirst') === true, "First!");
+  ok($('#node-1-2').treegrid('isFirst') === false, "Not First!");
+});
+
 test("isRoot()", function() {
   ok($('#node-1').treegrid('isRoot') === true, "Root test!");
   ok($('#node-1-2').treegrid('isRoot') === false, "Not Root test!");
@@ -115,7 +132,7 @@ test("Alphanumeric id", function() {
 });
 
 test("getAllNodes", function() {
-  equal($('#tree-1').treegrid('getAllNodes').length, 11, "11");
+  equal($('#tree-1').treegrid('getAllNodes').length, 12, "12");
 });
 
 test("isNode", function() {

treegrid.jquery.json

@@ -11,7 +11,7 @@
         "tree",
         "ui"
     ],
-    "version": "0.1.6",
+    "version": "0.1.7",
     "title": "TreeGrid plugin for jQuery",
     "author": {
         "name": "Pomazan Max"