Kajonczyk Roman Discord Server

[ironkajman]

No description provided.

[IamMK]

<div class="container">
       <p>Join the Academy!</p>



       <div class="bothCentered">

Takie odstępy są potrzebne?

[IamMK]

<label for="LandingEmail">Type Your Email Here</label><br><input type="text" id="LandingEmail" placeholder="E-Mail">
                 <br> <br>

Staraj się pisać jak najmniej znaczników br. Nie służą one do tego żeby pozycjonować elementy, tylko do załamania tekstu(Chociaż zazwyczaj używa się do tego znacznika p)

[IamMK]

console.log(tab);

Staraj się unikać zostawiania console logów w kodzie jeśli nie są konieczne ;)

[IamMK]

const tab = [];

staraj się prawidłowo nazywać zmienne. tab nic mi nie mówi, co robi ani za co odpowiada

[IamMK]

send.addEventListener("click", (e) => {
    if (input.value.length > 1) {
        tab.push(input.value);
        console.log(tab);
        input.value = "";
    }

window.addEventListener("keydown", (e) => {
    if (e.keyCode == 13 && input.value.length > 0) {
        tab.push(input.value);
        console.log(tab);
        input.value = "";
    }
})

te metody robią niemal to samo. Nie lepiej zrobić jedną?

[IamMK]

send.addEventListener("click", (e) => {
    if (input.value.length > 1) {
        tab.push(input.value);
        console.log(tab);
        input.value = "";
    }

e nie jest użyte więc nie ma sensu go definiować

[IamMK]

Staraj się unikać jak ognia stylowania po tagach w css ;) używaj klas

[ironkajman]

Dzięki wielkie za komentarze. W następnych projektach postaram się uwzględnić to co powiedziałeś