Unordered List, List Item, JS / HTML Question

Question

I found a few posts referencing something similar but I could not get it working..

Simply I am trying to create a List as shown below.

<ul>
    <li><a href="https://www.google.com">foo</a> </li>
    <li><a href="https://www.google.com">bar</a></li>
</ul>

However when generating it through JS I get one of the following.

<ul>
    <li>
        <a href="https://www.google.com"></a>
        foo
    </li>
    <li>
        <a href="https://www.google.com"></a>
        bar
    </li>
</ul>

OR

    <ul>
    <li>
        foo
        <a href="https://www.google.com"></a>
    </li>
    <li>
        bar
        <a href="https://www.google.com"></a>
    </li>
</ul>

With the JS code provided.

function addListItem(id, str, url) {

var list = document.getElementById(id);

var el = document.createElement("LI");

var text = document.createTextNode(str);

if (url !== "") {

    var a = document.createElement("A");

    a.setAttribute("href", url);

    el.appendChild(text);

    el.appendChild(a);

} else {

    el.appendChild(text);

}

list.appendChild(el);

}

How would I go about doing this?

Tyler Roper · Accepted Answer · 2018-10-23 04:06:40Z

The Issue

Take a look at these two lines:

el.appendChild(text);
el.appendChild(a);

First, you append the text...

<li>
    foo
</li>

Then you append the <a>...

<li>
    foo
    <a href="https://www.google.com"></a>
</li>

The Solution

What you need to do instead, is append your text to your <a> first:

a.appendChild(text);

<a href="https://google.com">foo</a>

Then append that to your <li>:

el.appendChild(a);

<li><a href="https://google.com">foo</a></li>

Corrected version:

function addListItem(id, str, url) {
  var list = document.getElementById(id);
  var el = document.createElement("LI");
  var text = document.createTextNode(str);
  
  if (url !== "") {
    var a = document.createElement("A");
    a.setAttribute("href", url);
    a.appendChild(text);
    el.appendChild(a);
  } else {
    el.appendChild(text);
  }

  list.appendChild(el);
}

addListItem("list", "foo", "https//google.com");

<ul id="list"></ul>

gotnull · Accepted Answer · 2018-10-23 03:59:25Z

1

Try this:

var a = document.createElement('a');
a.setAttribute('href', 'foo.com');
a.appendChild(document.createTextNode('bar'));
console.log(a);

answered Oct 23, 2018 at 3:59

gotnull

27.2k23 gold badges143 silver badges207 bronze badges

Comments

Neil.Work · Accepted Answer · 2018-10-23 04:00:23Z

1

The text is a child of the a tag, not the li tag.

a.appendChild(text);

answered Oct 23, 2018 at 4:00

Neil.Work

1,0258 silver badges9 bronze badges

Collectives™ on Stack Overflow

Unordered List, List Item, JS / HTML Question

3 Answers 3

The Issue

The Solution

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

The Issue

The Solution

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related