Jquery Passing Table Rows Between Functions

Question

I have a simple grid layout I wrote using jQuery. I have one problem. I want to allow only one edit row at a time. So I keep track of current editing row and reset it on next edit button is clicked. Still when I click edit for next time it allows me edit. completeEdit works fine cause I am also calling it from cancel button by passing current row.

 var currentRowEdit =null;

   $(tableid + ".edit").live('click', function(event) {
            currentRowEdit = $(this).parent().parent();
            editRow(currentRowEdit);
    });


function editRow(row){
        if(currentRowEdit!=null){
            completeEdit(currentRowEdit);               
        }              
           $(row).find(".save").show();
           $(row).find(".cancel").show();
           $(row).find(".edit").hide();

    }

function completeEdit(row){             
         $(row).find(".save").hide();
          $(row).find(".cancel").hide();
         $(row).find(".edit").show();              
    }

thecodeparadox · Accepted Answer · 2012-09-13 20:03:11Z

1

function completeEdit(row){
   $(row).find(".save").hide();
   $(row).find(".cancel").hide();
   $(row).find(".edit").show();   
   currentRowEdit = null; // reset the currentRowEdit
}

According to comment

Your code is not hiding the the .save and .cancel button. Because, if you look at you code flow will notice, after the completeEdit() function you're using .show() method for those button.

As a result, your buttons hide when completeEdit() done but after that when rest of the code of editRow() executed, then those buttons again become visible.

edited Sep 13, 2012 at 20:03

answered Sep 13, 2012 at 19:56

thecodeparadox

87.2k22 gold badges143 silver badges164 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

thecodeparadox Over a year ago

@SoneshDabhi please check the update and hope you will find clues.

Anoop · Accepted Answer · 2012-09-13 20:01:42Z

0

I believe you are calling completeEdit and editRow on same row. you should call completeEdit on previous and editRow on new

 var currentRowEdit =null;

   $(tableid + ".edit").live('click', function(event) {
            completeEdit(currentRowEdit);
            currentRowEdit = $(this).parent().parent();
            editRow(currentRowEdit);
    });


function editRow(row){



           $(row).find(".save").show();
           $(row).find(".cancel").show();
           $(row).find(".edit").hide();

    }

function completeEdit(row){
         if(row==null){
           return;

        }
         $(row).find(".save").hide();
          $(row).find(".cancel").hide();
         $(row).find(".edit").show();    

    }

answered Sep 13, 2012 at 20:01

Anoop

23.2k10 gold badges65 silver badges79 bronze badges

1 Comment

Pit Digger Over a year ago

Thanks . I made that change . But still the problem remains .

Collectives™ on Stack Overflow

Jquery Passing Table Rows Between Functions

2 Answers 2

According to comment

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

According to comment

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related