CODEWITHSUNDEEP
javascriptjqueryhtmlcss
Martin AJ
Martin AJ

Reputation: 6697

How to check/remove an element?

$(function(){

  $("#tags input").on({
    focusout : function() {
      var txt = this.value.replace(/[^a-z0-9\+\-\.\#]/ig,''); // allowed characters
      if(txt) $("<span/>", {text:txt.toLowerCase(), insertBefore:this});
      this.value = "";
    },
    keyup : function(ev) {
      // if: comma|enter (delimit more keyCodes with | pipe)
      if(/(188|13|32)/.test(ev.which)){
        $(this).focusout();
      } else if (/(8)/.test(ev.which) && this.value == '' && $(this).prev("span").css('background-color')=="rgb(225, 0, 0)") {
        $(this).prev("span").remove();
      } else if (/(8)/.test(ev.which) && this.value == '') {
        $(this).prev("span").css('background-color', 'Red');
      }
    }
  });
  $('#tags').on('click', 'span', function() {
     $(this).remove(); 
  });

});
#tags{
  float:right;
  border:1px solid #ccc;
  padding:5px;
  font-family:Arial;
}
#tags > span{
  cursor:pointer;
  display:block;
  float:right;
  color:#3e6d8e;
  background:#E1ECF4;
  padding:5px;
  padding-right:25px;
  margin:4px;
}
#tags > span:hover{
  opacity:0.7;
}
#tags > span:after{
 position:absolute;
 content:"×";
 border:1px solid;
 padding:2px 5px;
 margin-left:3px;
 font-size:11px;
}
#tags > input{
  direction: rtl;
  background:#eee;
  border:0;
  margin:4px;
  padding:7px;
  width:auto;
}
.autocomplete{
  display:none;
}
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>

<div id="tags">
  <span>php</span>
  <span>html</span>
  <input type="text" value="" placeholder="Add a tag" />
  <div class="autocomplete"></div>
</div>

I want to remove a tag either by pressing backspace button (when the input is focused) or by clicking on that tag. Now I'm talking about the first option. Also I need to set it a red background color first (as "are you sure?" concept). Ok as you see, my fiddle sets it a red background color, but it doesn't remove it. what's wrong?

Upvotes: 0

Views: 85

Answers (2)

A. Wolff
A. Wolff

Reputation: 74420

You'd have better to toggle a class and check for it, e.g:

$(function() {

  $("#tags input").on({
    focusout: function() {
      var txt = this.value.replace(/[^a-z0-9\+\-\.\#]/ig, ''); // allowed characters
      if (txt) $("<span/>", {
        text: txt.toLowerCase(),
        insertBefore: this
      });
      this.value = "";
    },
    keyup: function(ev) {
      // if: comma|enter (delimit more keyCodes with | pipe)
      if (/(188|13|32)/.test(ev.which)) {
        $(this).focusout();
      } else if (/(8)/.test(ev.which) && this.value == '' && $(this).prev("span").hasClass('toRemove')) { //<< check for class
        $(this).prev("span").remove();
      } else if (/(8)/.test(ev.which) && this.value == '') {
        $(this).prev("span").addClass('toRemove'); //<< add class
      } else {
        $(this).prevAll('.toRemove').removeClass('toRemove'); //<< remove class on keyup
      }
    }
  });
  $('#tags').on('click', 'span', function() {
    $(this).remove();
  });

});
#tags {
  float: right;
  border: 1px solid #ccc;
  padding: 5px;
  font-family: Arial;
}
#tags > span {
  cursor: pointer;
  display: block;
  float: right;
  color: #3e6d8e;
  background: #E1ECF4;
  padding: 5px;
  padding-right: 25px;
  margin: 4px;
}
#tags > span:hover {
  opacity: 0.7;
}
#tags > span:after {
  position: absolute;
  content: "×";
  border: 1px solid;
  padding: 2px 5px;
  margin-left: 3px;
  font-size: 11px;
}
#tags > input {
  direction: rtl;
  background: #eee;
  border: 0;
  margin: 4px;
  padding: 7px;
  width: auto;
}
.autocomplete {
  display: none;
}
#tags > span.toRemove {
  background-color: red;
}
<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script>
<div id="tags">
  <span>php</span>
  <span>html</span>
  <input type="text" value="" placeholder="Add a tag" />
  <div class="autocomplete"></div>
</div>

Upvotes: 1

Marcel Wasilewski
Marcel Wasilewski

Reputation: 2679

I built in a counter for you that checks the status of your keypress. If you press once the counter is 1 and if you press the second time it removes the element and is 0 again. I hope thats what you wanted. Check this:

$(function() {

  counter = 0;

  $("#tags input").on({
    focusout: function() {
      var txt = this.value.replace(/[^a-z0-9\+\-\.\#]/ig, ''); // allowed characters
      if (txt) $("<span/>", {
        text: txt.toLowerCase(),
        insertBefore: this
      });
      this.value = "";
      $("#tags span").css('background-color', '#E1ECF4');
      counter = 0;
    },
    keyup: function(ev) {
      // if: comma|enter (delimit more keyCodes with | pipe)
      if (/(188|13|32)/.test(ev.which)) {
        $(this).focusout();
      } else if (/(8)/.test(ev.which) && this.value == '') {
        counter = counter + 1;
        if (counter == 1) {
          $(this).prev("span").css('background-color', 'Red');
        } else {
          $(this).prev("span").remove();
          counter = 0;
        }
      }
    }
  });
  $('#tags').on('click', 'span', function() {
    $(this).remove();
      $("#tags span").css('background-color', '#E1ECF4');
      counter = 0;
  });

});

https://jsfiddle.net/4swtduae/7/

Upvotes: 1

Related Questions