can you guys help me with the order for the dom?

Question

I have a problem with undefined property I added the first part of my js I have a document.readystate witch suppose to help with the Dom not reading fills but I still get undefined

if (document.readyState == `loading`) {
  document.addEventListener(`DOMContentLoaded`, ready);
} else {
  ready
}

function ready() {
  var hiddentwo = document.getElementById(`sectiontop`)
  var hidden = document.getElementById(`sectionone`)
  var hiddentwo = document.getElementById(`sectiontop`)
  console.log(hiddentwo)
  const openModal = function() {
    hidden.classList.remove(`hidesection`);
  };
  const closeModal = function() {
    hidden.classList.add('hidden');
  };
  const closeModal1 = function() {
    hiddentwo.classlist.remove(`hidesection1`)


  };

  const closeModal11 = function() {
    hiddentwo.classlist.add(`hidesection1`)


  };
  window.onload = function() {
    hiddentwo.classlist.remove('hidesection1')
  };

};

.hidesection1 {
  display: none;
}

<section id="sectiontop" class="hidesection1">

_What_ is `undefined`? Elements don’t have a property called `classlist`; you mean [`classList`](https://developer.mozilla.org/en-US/docs/Web/API/Element/classList). Further, is your ` — Sebastian Simon, Mar 21 '21 at 13:02
@SebastianSimon, I meant quotation marks. Seemed lack of quotes. — s.kuznetsov, Mar 21 '21 at 13:13
@connexo i did what you said but still the error still there — 37bandiko, Mar 21 '21 at 13:15
@connexo i dont understand how readystate works i just was following a tutorial and told me to do that in the beginning of the script — 37bandiko, Mar 21 '21 at 13:17
@s.kuznetsov No, the quotes weren’t lacking. The quotes are correct. — Sebastian Simon, Mar 21 '21 at 13:20
https://developer.mozilla.org/en-US/docs/Web/API/Document/readyState — connexo, Mar 21 '21 at 13:21
@connexo ty for telling but im not using hiddeone in my code i need to delete that — 37bandiko, Mar 21 '21 at 13:34
@SebastianSimon, I also meant that the quotes are correct. At first it seemed to me that there were no quotes at all. — s.kuznetsov, Mar 21 '21 at 13:46
Is your question answered? If not, please comment accordingly. If yes, please pick an answer. — connexo, Mar 22 '21 at 08:54

connexo · Answer 1 · 2021-03-21T13:23:34.383

Remove this block in your code:

if  (document.readyState == `loading`)
    {document.addEventListener(`DOMContentLoaded`,ready);}
else{ready}

and replace it with either

document.addEventListener(`DOMContentLoaded`, ready);

or use the ready state change event (which requires checking the state inside your ready function):

document.addEventListener(`readystatechange`, ready);

// and in ready():
function ready() {
    if (document.readyState === 'interactive') {
        // your code from ready function body here
    }
};

You always use either method, never both.

score 0 · Answer 2 · answered Mar 21 '21 at 18:36

0

as @Sebastian Simon and @ conexo pointed out hiddentwo.classlist.remove must be hiddentwo.classList.remove. Case matters in Javascript.

answered Mar 21 '21 at 18:36

37bandiko

9
4

can you guys help me with the order for the dom?

2 Answers2