2

I'm trying to get a different width for each li element inside ul, but as soon as I add a larger element all the li get the same width. I want the lis have the same width as the text length.

My HTML:

<div>
 <ul>

 </ul>
</div>
<textarea></textarea>
<button>Send</button>

My CSS:

div{
    overflow-y: scroll;
    overflow-x: hidden;
    height: 400px; 
    width:400px;
    border:1px solid black;
}
div ul {
    display: table;
    list-style: none;
}
div ul li{
    display: block;
    background-color: #b70505;
    color: white;
    font-weight: 600;
    font-size: 0.8em;
    border-radius: 25px;
    margin-top: 5px;
    padding: 10px;
    max-width:200px;
    word-wrap:break-word;
}

My JS:

var box = document.getElementsByTagName('div')[0];
var newText = document.getElementsByTagName('textarea')[0];
var button = document.getElementsByTagName('button')[0];

button.addEventListener('click', function(e){
    e.preventDefault();
    if(newText.value !== ''){
        var Newitem = document.createElement('li');
        Newitem.innerHTML = newText.value;
        box.getElementsByTagName('ul')[0].appendChild(Newitem);
        box.scrollTop = box.scrollHeight;
    }
});

https://jsfiddle.net/hdr60zmb/

What am I doing wrong?

Roberto Caboni
  • 7,252
  • 10
  • 25
  • 39
epselonzero
  • 51
  • 1
  • 6

1 Answers1

2

In order to make list items width dependent from text, remove display attributes from ul and li and just set float: left to li elements. If you want to have each list item in a new line, add clear: left.

CSS:

div ul {
    list-style: none;
}

div ul li {
    float: left;
    clear: left;
    background-color: #b70505;
    color: white;
    font-weight: 600;
    font-size: 0.8em;
    border-radius: 25px;
    margin-top: 5px;
    padding: 10px;
    max-width: 200px;
    word-wrap: break-word;
}

Demo: https://jsfiddle.net/rwkb03xc/

Remember about clearfixing the ul, to keep height of that element. The simplest solution is to add overflow: auto to ul. More on this: What methods of ‘clearfix’ can I use?

EMiDU
  • 654
  • 1
  • 7
  • 24