On click event

Question

Witam,

Chcę, aby div z paskiem ładowania pokazywałsię po naciśnięciu przycisku "NEXT". Niestety nic się nie pokazuje :(. Skrypt JS do pokazywania diva mam w osobnym pliku .js. Poniżej kod:

<!-- Progress bar -->
             <div id="progress-bar" class="progress invisible">
                    <div id="theBar" class="progress-bar progress-bar-info progress-bar-striped active" role="progressbar" aria-valuenow="0" aria-valuemin="0" aria-valuemax="100" style="width: 0%; height:25px">
                        0%
                    </div>
                    <div>
                        Connexting...
                    </div>
            </div>
        <br/>
    
            
            
            
          <!-- NEXT -->  
            
            <div class= "text-center">
                
                <button id="next" type="button" class="btn btn-secondary" onclick="progressbar();">NEXT</button>
                <br/>
                <span id="terms">By clicking next you agree with our terms and conditions and privacy policy</span>
            </div>

A tutaj skrypt w JS(używam bootstrapa):

function progressbar() {
document.getElementById("progress-bar").classList.remove('invisible');

}

Co jest nie tak? Dodam że skrypt zaimportowałem na początku dokuentu HTML :

    <script type="text/javascript" scr="progress-bar.js"></script>

Aczkolwiek inny skrypt działą mi bez importu...

 

Dzięki za pomoc

Comments

F12 --> console

Jakieś błędy tam są?

Bo kod jest niby dobry (pomijam semantykę HTML). Oprócz tego, że JS powinien być zawsze na końcu body, a nie gdzieś na początku pliku.

I zamiast onclick(), warto używać: https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/addEventListener

Answer 1

Może to ten średnik:

 

 <button id="next" type="button" class="btn btn-secondary" onclick="progressbar();">NEXT</button>

Tak nawiasem mówiąc, to takie dodawanie zdarzenia raczej nie jest zalecane:p.

Comments

O jaki średnik chodzi? :D Bo mój kod jest taki sam jak Twój.

Jaką polecasz zatem metodę? Ja robię to pierwszy raz więc nie wiem co jest najlepsze, ale pewnie to zmienię.

Dzięki za odpowiedź

---

Ja tam ekspertem nie jestem, ale poczytaj o addEventListener.

Zastanawiam się, czy po usunięciu tego średnika po nawiasach przy nazwie metody w onclicku nie będzie działać, ale możliwe, że to nie ma znaczenia:P. Po prostu sam nigdy nei widziałem, żeby tam był w zapisie, ale sam nie sprawdzałem.