Change to using jquery

Asked Apr 19 '18 at 15:39

Active Apr 19 '18 at 16:40

Viewed 22 times

I have problem with my code, I want change to by id

this is my PHP

<?php
    foreach($brands->result() as $row){
?>
    <tr id ="<?=$row->brand_id?>">
        <td><?=$row->brand_id;?></td>
        <td class="editableName"><?=$row->name_brand;?></td>
        <td class="editableDesc"><?=$row->description;?></td>
        <td>
            <button id="edit" class="form-control">Edit</button>
            <a href="<?=base_url('Admin/brands/delete')?>/<?=$row->brand_id;?>"><button class="form-control">Delete</button>
        </td>
    </tr>
<?php 
    } 
?>

This is my JQuery

$('#edit').click(function () {
    $(this).parents('tr').find('td.editableName').each(function() {
        var html = $(this).html();
        var input = $('<input name="name" class="form-control" type="text" />');
        input.val(html);
        $(this).html(input);
    });
    $(this).parents('tr').find('td.editableDesc').each(function() {
        var html = $(this).html();
        var input = $('<input name="description" class="form-control" type="text" />');
        input.val(html);
        $(this).html(input);
    });
});

i am using that jquery script its only work on 1st row, the nth row doesn't work

edited Apr 19 '18 at 16:40

Ignacio Ara

2,476
2
26
37

asked Apr 19 '18 at 15:39

Tukang Sihir

2

You're re-using `id` values in your HTML. `id` is supposed to be unique. The jQuery selector is only finding the *first* one because there shouldn't be more. You should probably use a `class` value instead of an `id`. – David Apr 19 '18 at 15:40
You should change button selector on class or add onclick for each button. You cannot use same id names. – Anastasia Bashkirtseva Apr 19 '18 at 15:44
so i change the `` to `` like that? – Tukang Sihir Apr 19 '18 at 15:44
1

change `` to `` and on js use `$('.edit').click(` – Mohamed-Yousef Apr 19 '18 at 15:46
thanks @Mohamed-Yousef – Tukang Sihir Apr 19 '18 at 15:57
You're welcome @TukangSihir :-) – Mohamed-Yousef Apr 19 '18 at 15:59

Change to using jquery

0 Answers0