How to renumber items when one of them is removed?-CodePudding

I have a use-case where input controls are listed and a remove button. The user has the option to remove any of the item in the list. I need to be able to check that there is more then one item in the list before I can remove the item. I also need to renumber the div, span and input control. So if a user removes id=1 I want to remove that item and they renumber then starting with 1.

function RemoveRow(id) {
  if (id > 0) {
    let rowCount = $('[id^=newrowItem_]').length
    if (rowCount != 1) {
      let removeData = `#newrowItem_${id}`;
      $(removeData).remove();
      // Re-number
      let inputdatagripPanel = $("#inputDataGrippanel");
      var inputdatagripForm = inputdatagripPanel.find(".form-group");
      let fieldItemid = 1;
     // $(inputdatagripForm).each((index, item) => {
     //   console.log(item);
     //   let fieldItem = `#fieldrowItem_${fieldItemid}`;
     //   item.attr('id', fieldItem);
     //   id  ;
     // });
    } else {
      let fieldItem = `#fieldrowItem_${id}`;
      $(fieldItem).addClass('error');
    }
  }
}

<link href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.2.1/css/all.min.css" rel="stylesheet" />
<script type="text/javascript" src="//cdnjs.cloudflare.com/ajax/libs/jquery/3.4.1/jquery.js"></script>
<div id="inputDataGrippanel" >
  <div id="inputDataGrippanel" >
    <label>Rows</label>
    <div  id="newrowItem_1">
      <div >
        <input type="text" id="fieldrowItem_1"  placeholder="Row 1" data-value="0" required="" value="a">
        <span id="spanrowItem_1"  style="cursor:pointer;" onclick="RemoveRow(1)">
          <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
    <div  id="newrowItem_2">
      <div >
        <input type="text" id="fieldrowItem_2"  placeholder="Row 2" data-value="0" required="" value="b">
        <span id="spanrowItem_2"  style="cursor:pointer;" onclick="RemoveRow(2)">
          <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
    <div  id="newrowItem_3">
      <div >
        <input type="text" id="fieldrowItem_3"  placeholder="Row 3" data-value="0" required="" value="c">
        <span id="spanrowItem_3"  style="cursor:pointer;" onclick="RemoveRow(3)">
          <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
    <div  id="newrowItem_4">
      <div >
        <input type="text" id="fieldrowItem_4"  placeholder="Row 4" data-value="0" required="" value="d">
        <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
    <div  id="newrowItem_5">
      <div >
        <input type="text" id="fieldrowItem_5"  placeholder="Row 5" data-value="0" required="" value="e">
        <span id="spanrowItem_5"  style="cursor:pointer;" onclick="RemoveRow(5)">
          <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
  </div>

CodePudding user response：

I have fixed a missing span in your HTML, modified the code that calls RemoveRow to be dynamic and added the logic that renumbers

function RemoveRow(id) {
  let divs = document.querySelectorAll('div.form-group');
  if ((id > 0) && (id <= divs.length)) {
    let rowCount = $('[id^=newrowItem_]').length
    if (rowCount != 1) {
      let removeData = `#newrowItem_${id}`;
      $(removeData).remove();
      // Re-number
      /*let inputdatagripPanel = $("#inputDataGrippanel");
      var inputdatagripForm = inputdatagripPanel.find(".form-group");
      let fieldItemid = 1;*/
      let index = 1;
      divs = document.querySelectorAll('div.form-group');
      for (let div of divs) {
          div.id = `newrowItem_${index}`;
          div.querySelector('input').id = `fieldrowItem_${index}`;
          div.querySelector('span').id = `spanrowItem_${index}`;
          index  ;
      }
     // $(inputdatagripForm).each((index, item) => {
     //   console.log(item);
     //   let fieldItem = `#fieldrowItem_${fieldItemid}`;
     //   item.attr('id', fieldItem);
     //   id  ;
     // });
    } else {
      let fieldItem = `#fieldrowItem_${id}`;
      $(fieldItem).addClass('error');
    }
  }
}

<link href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.2.1/css/all.min.css" rel="stylesheet" />
<script type="text/javascript" src="//cdnjs.cloudflare.com/ajax/libs/jquery/3.4.1/jquery.js"></script>
<div id="inputDataGrippanel" >
  <div id="inputDataGrippanel" >
    <label>Rows</label>
    <div  id="newrowItem_1">
      <div >
        <input type="text" id="fieldrowItem_1"  placeholder="Row 1" data-value="0" required="" value="a">
        <span id="spanrowItem_1"  style="cursor:pointer;" onclick="RemoveRow(parseInt(this.id.split('_')[1]))">
          <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
    <div  id="newrowItem_2">
      <div >
        <input type="text" id="fieldrowItem_2"  placeholder="Row 2" data-value="0" required="" value="b">
        <span id="spanrowItem_2"  style="cursor:pointer;" onclick="RemoveRow(parseInt(this.id.split('_')[1]))">
          <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
    <div  id="newrowItem_3">
      <div >
        <input type="text" id="fieldrowItem_3"  placeholder="Row 3" data-value="0" required="" value="c">
        <span id="spanrowItem_3"  style="cursor:pointer;" onclick="RemoveRow(parseInt(this.id.split('_')[1]))">
          <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
    <div  id="newrowItem_4">
      <div >
        <input type="text" id="fieldrowItem_4"  placeholder="Row 4" data-value="0" required="" value="d">
        <span id="spanrowItem_4"  style="cursor:pointer;" onclick="RemoveRow(parseInt(this.id.split('_')[1]))">
        <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
    <div  id="newrowItem_5">
      <div >
        <input type="text" id="fieldrowItem_5"  placeholder="Row 5" data-value="0" required="" value="e">
        <span id="spanrowItem_5"  style="cursor:pointer;" onclick="RemoveRow(parseInt(this.id.split('_')[1]))">
          <i  style="color:#CDCDCD"></i>
        </span>
      </div>
    </div>
  </div>