ooop
03-05-2007, 08:18 PM
1) Script Title: DHTML Window Widget
2) Script URL (on DD): http://www.dynamicdrive.com/dynamicindex8/dhtmlwindow/index.htm
3) Describe problem: Not so much bugs as some things overlooked...
----- dhtmlwindow.js
line 25 seems out of place and not needed: it looks like it closes a div that doesn't exist
- to illustrate
- following are lines 20 to 25
-- expanded with my comments:
domwindowdata='<div class="drag-handle">'
// first div opened (drag-handle)
domwindowdata+='DHTML Window <div class="drag-controls"><img src="'+this.imagefiles[0]+'" title="Minimize" /><img src="'+this.imagefiles[1]+'" title="Close" /></div>'
// second div (drag-controls) opened and closed - first div still open (drag-handle)
domwindowdata+='</div>'
// first div closed (drag-handle) - no div is open
domwindowdata+='<div class="drag-contentarea"></div>'
// third div (drag-contentarea) opened and closed - no div is open
domwindowdata+='<div class="drag-statusarea"><div class="drag-resizearea" style="background: transparent url('+this.imagefiles[3]+') top right no-repeat;"> </div></div>'
// two divs (drag-statusarea and drag-resizearea) are opened and closed - no div is open
domwindowdata+='</div>'
// [ line 25 ] so what div does this close?
------------------------------------------------
AND
----- images: min.gif and resize.gif
speak about BLOAT - these two images use an indexing of colors that go far beyond the scope of what's called for:
min.gif - 872 bytes
resize.gif - 859 bytes
when each could be easily around 80 to 90 bytes if properly indexed--- i know we're talking only bytes - but still - all the other control images are around 125 bytes - you think someone would have noticed the difference in these two images...
2) Script URL (on DD): http://www.dynamicdrive.com/dynamicindex8/dhtmlwindow/index.htm
3) Describe problem: Not so much bugs as some things overlooked...
----- dhtmlwindow.js
line 25 seems out of place and not needed: it looks like it closes a div that doesn't exist
- to illustrate
- following are lines 20 to 25
-- expanded with my comments:
domwindowdata='<div class="drag-handle">'
// first div opened (drag-handle)
domwindowdata+='DHTML Window <div class="drag-controls"><img src="'+this.imagefiles[0]+'" title="Minimize" /><img src="'+this.imagefiles[1]+'" title="Close" /></div>'
// second div (drag-controls) opened and closed - first div still open (drag-handle)
domwindowdata+='</div>'
// first div closed (drag-handle) - no div is open
domwindowdata+='<div class="drag-contentarea"></div>'
// third div (drag-contentarea) opened and closed - no div is open
domwindowdata+='<div class="drag-statusarea"><div class="drag-resizearea" style="background: transparent url('+this.imagefiles[3]+') top right no-repeat;"> </div></div>'
// two divs (drag-statusarea and drag-resizearea) are opened and closed - no div is open
domwindowdata+='</div>'
// [ line 25 ] so what div does this close?
------------------------------------------------
AND
----- images: min.gif and resize.gif
speak about BLOAT - these two images use an indexing of colors that go far beyond the scope of what's called for:
min.gif - 872 bytes
resize.gif - 859 bytes
when each could be easily around 80 to 90 bytes if properly indexed--- i know we're talking only bytes - but still - all the other control images are around 125 bytes - you think someone would have noticed the difference in these two images...