modal box na dwa buttony

Question

Witam. :)
Mam pewien problem. Zrobiłem jeden modal box dla przycisku "Zaloguj" problem w nim jest taki ,że tworzy przerwę pomiędzy przyciskiem rejestruj. A przycisk rejestruj w ogóle nie reaguje na js. Css jest zrobiony poprawnie. Próbowałem zrobić nowy js z lekko zmienionymi wartości dla rejestruj lecz nic to nie zmienia. :/

Tutaj kod z zaloguj: https://codepen.io/Kondzik/pen/eMGXpP

a tutaj do js rejestruj

// Zadelkarowanie modal jako simplemodal
var modal = document.getElementById('simplemodal1');


// OZadelkarowanie modalBtn jako przycisku otwierajacego

var modalBtn = document.getElementById('modalBtnSign');


// Zadelkarowanie modal jako przycisku zamykajacego
var closeBtn = document.getElementsByClassName('closeBtn')[0];

// Nadanie otwarcia

modalBtnSign.addEventListener('click', openModal);

//	Nadanie zamknięcia
closeBtn.addEventListener('click', closeModal);

// Nadanie zamknieica poza obszarem
window.addEventListener('click', outsideClick);

// Funkcja otwarcia
function openModal(){
	modal.style.display= 'block';
}

// Funkcja zamknięcia
function closeModal(){
	modal.style.display= 'none';
}

// Funkcja zamknięcia poza obszarem
function outsideClick(e){
	if(e.target == modal){
		modal.style.display = 'none';
	}
}

 

Comments

Niepoprawny język bloczka z kodem.

---

Poprawione, ale proszę na przyszłość o tym pamiętać. 

---

Przepraszam wrzucane na szybko :)

Answer 1

// Zadelkarowanie modal jako simplemodal
var modal = document.getElementById('simplemodal');


// OZadelkarowanie modalBtn jako przycisku otwierajacego

var modalBtn = document.getElementById('modalBtnLog');


// Zadelkarowanie modal jako przycisku zamykajacego
var closeBtn = document.getElementsByClassName('closeBtn')[0];

// Nadanie otwarcia

modalBtnLog.addEventListener('click', openModal);

//	Nadanie zamknięcia
closeBtn.addEventListener('click', closeModal);

// Nadanie zamknieica poza obszarem
window.addEventListener('click', outsideClick);

// Funkcja otwarcia
function openModal(){
	modal.style.display= 'block';
}

// Funkcja zamknięcia
function closeModal(){
	modal.style.display= 'none';
}

// Funkcja zamknięcia poza obszarem
function outsideClick(e){
	if(e.target == modal){
		modal.style.display = 'none';
	}
}


// Zadelkarowanie modal jako simplemodal
var modal1 = document.getElementById('simplemodal1');


// OZadelkarowanie modalBtn jako przycisku otwierajacego

var modalBtn1 = document.getElementById('modalBtnSign');


// Zadelkarowanie modal jako przycisku zamykajacego
var closeBtn1 = document.getElementsByClassName('closeBtn1')[0];

// Nadanie otwarcia

modalBtnSign.addEventListener('click', openModal1);

//	Nadanie zamknięcia
closeBtn1.addEventListener('click', closeModal1);

// Nadanie zamknieica poza obszarem
window.addEventListener('click', outsideClick);

// Funkcja otwarcia
function openModal1(){
	modal1.style.display= 'block';
}

// Funkcja zamknięcia
function closeModal1(){
	modal1.style.display= 'none';
}

// Funkcja zamknięcia poza obszarem
function outsideClick(e){
	if(e.target == modal){
		modal1.style.display = 'none';
	}
}

Wiem że to jest frankie i reduntancja ale chciałem Cie nakierowac na problem.

Nie znam sie na js ale zrobiłem że działa.

Comments

Nie zrobiłes zmiennych do drugiego przycisku.

---

Były zrobione tylko niepoprawnie i to w innym pliku :D
Dziękuje bardzo :D wszystko śmiga :D

To teraz nasunęły mi się dwa pytania jeszcze. :)
1. Dlaczego teraz nie działa wyłączenie okna klikając poza obszarem. (W rejestruj był ten sam problem ale już sobie poradziłem)
2. Jak usunąć tą odległość między przyciskami? :)

---

Na pierwsze pytanie odpowiedziałeś sobie sam-zrób to samo co z tamtym przyciskiem.

---

outsideClick1<-- powinno być w js na samym dole zamiast ousideclick.

to załatwia 1 pytanie.

Edit: to nie do końca działa.

---

Dobra załatwiło problem :D dzięki :D

 

Działa tylko trzeba zamienić w dwóch miejscach. :D Jeszcze tylko jak to do siebie przybliżyć :/