Potential bug with BasicDialog? - Ext JS

#3 03-05-2007, 12:55 PM

Not sure why you'd want to do all that work yourself - you can use autoCreate:true to tell the Dialog to create all the HTML for you. Also why are you rendering the dialog to a UL element?
You might try using Firebug to compare the generated HTML of your approach vs what happens when you use autoCreate, or what's built when you just point it to a div and let it build all the child elements.

#4 03-05-2007, 01:04 PM

I'm new to Ext. Can you explain? Basically, I'm just creating an unordered list based on the data and making that the content of the dialog. It should actually be divElement because I tried placing it within a div to see if that made a difference but it didn't. I've switched over to JSON, so that bit actually look like this:

		var playlists = eval(response.responseText);
		for (var i = 0; i < playlists.length; i++){
			var listId = playlists[i]['listid'];
			var title = playlists[i]['title'];
			var opt = document.createElement('li');
			listElement.appendChild(opt);
			opt.setAttribute('listId',listId);
			opt.appendChild(document.createTextNode(title));
			opt.onclick = function () {
				addSong(this.getAttribute('listId'),currentSong['trackid']);
				dialog.hide();
			};
		}

#5 03-05-2007, 02:31 PM

This is one of the common uses for the Template functionality. Take a look at this blog post - it actually has examples that build lists like you're doing - note that it still uses the .33 syntax, so you'll have to replace things like YAHOO.ext with Ext. http://www.jackslocum.com/blog/2006/...-or-templates/

I would also recommend looking thru downloaded examples - you'll find more uses of templates and also wiring a single click handler, rather than one for each 'LI'. Once you have it working in a div, it should be simple to append it to a Dialog instead - note that you append to the dialog.body, not the dialog.