Remove element's class using only JavaScript (no jQuery)

Question

I'm trying to get an element by class name, and after, remove it's class.

I wish to accomplish that using JavaScript only, no jQuery.

Here is what I've tried already:

<script>
  var changeController = function() {
    var removeClass = document.getElementsByClassName("selected-hr")[0];
    removeClass.classList.remove("selected-hr");
  }
</script>

<div class="col col-25 selected-hr">
  <a class="tb-header-link" onclick="changeController()">ALL</a>
</div>

EDIT 1: Fixed typo at changeController()

onclick="changeController()"

Pointy
– Pointy

2017-05-02 16:17:49 +00:00
Commented May 2, 2017 at 16:17 — Pointy
– Pointy, Commented May 2, 2017 at 16:17
This works for me without any problem.

Yashar Aliabbasi
– Yashar Aliabbasi

2017-05-02 16:34:36 +00:00
Commented May 2, 2017 at 16:34 — Yashar Aliabbasi
– Yashar Aliabbasi, Commented May 2, 2017 at 16:34
onclick="return changeController();" professionalism..

Janen R
– Janen R

2017-05-02 16:35:55 +00:00
Commented May 2, 2017 at 16:35 — Janen R
– Janen R, Commented May 2, 2017 at 16:35
code working fine

surekha shelake
– surekha shelake

2017-05-02 17:52:00 +00:00
Commented May 2, 2017 at 17:52 — surekha shelake
– surekha shelake, Commented May 2, 2017 at 17:52

Community · Accepted Answer · 2020-06-20 09:12:55Z

1

Pass this into changeController(), use parentNode property to reference the div. Place the <script> after the your markup (HTML) to ensure DOM is loaded before running script. Preferably before the </body> closing tag.

Snippet

a {text-decoration:none;}
.selected-hr {background:tomato;}

<div class="col col-25 selected-hr">
  <a href='#/' class="tb-header-link" onclick="changeController(this)">ALL</a>
</div>

<script>
  function changeController(ele) {
    var parent = ele.parentNode;
    parent.classList.remove("selected-hr");
  }
</script>

edited Jun 20, 2020 at 9:12

CommunityBot

11 silver badge

answered May 2, 2017 at 16:45

zer00ne

45.2k6 gold badges50 silver badges82 bronze badges

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

Comments

Quentin · Accepted Answer · 2017-05-02 16:19:00Z

0

Just mentioning the name of a variable does nothing (even if the variable points to a function).

You have to call it if you want to do anything with it. Add ().

onclick="changeController()"

answered May 2, 2017 at 16:19

Quentin

948k136 gold badges1.3k silver badges1.4k bronze badges

3 Comments

GreetingsFriend Over a year ago

Ok, that was a typo. I already fixed it, but the code is still not working.

Quentin Over a year ago

@GreetingsFriend — Given that fix, it works for me: jsbin.com/tubuduh/1/edit?html,css,output

GreetingsFriend Over a year ago

I just checked the console. changeController is not defined. The script is on a nested view just on top of the html(using angular-ui router). Where should i place the script?

Collectives™ on Stack Overflow

Remove element's class using only JavaScript (no jQuery)

2 Answers 2

Snippet

Comments

3 Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Snippet

Comments

3 Comments

Related