26 Jan 2007 6:20 AM #1
Bug in Dialog Render
I noticed that when I display a dialog in FF the bottom "close" button gets cut off. It is fine in MIE.
cut off bottom in FF 22.214.171.124
looks fine in MIE 7.0.5
when you resize the window in FF it goes away. a force-refresh brings it back.
hopefully it isn't something I am doing incorrectly from my end.
here is a live example: http://www.gabbr.com/sandbox/yahooui/yahoodialog8.php
26 Jan 2007 6:58 AM #2
- Join Date
- Mar 2007
- Notts/Redwood City
- Vote Rating
You're specifying autoCreate:true, but passing an existing div name - that will result in duplicated ids, and we know that's wrong!
Remove the <div id="dialog"... element.
Then use dial.header to get the header Element to update and dial.body to get the body element to update.
Your probs could be to do with that.
3 Feb 2007 6:04 PM #3
I even specified autocreate: false to no avail
does anyone else know why this problem occurs?
i still havent been able to solve it.. any help would be appreciated
3 Feb 2007 8:15 PM #4
Dialog should be rendered to the body, not nested inside another div. Try moving the div outside the wrapper div.
4 Feb 2007 11:08 AM #5Originally Posted by tryanDLS
playing around with the width/height/minwidth/height doesn't change it either.
I dumped the <div id="dialog"> -- to no avail.
I followed the "Hello World" dialog in the documentation as well and still get the error.
I even made a "vanilla" version -- I stripped all the div's (except for the dialog divs) -- and still have the same display issue/bug:
4 Feb 2007 12:52 PM #6
Try adding the footer div to your dialog layout e.g
4 Feb 2007 1:51 PM #7
4 Feb 2007 2:11 PM #8
This is fixed in my local trunk. The problem is the buttons are "float:right" and therefore don't take up space (some times).
I fixed it by adding this line right before the call to syncBodyHeight() (near the end) in BasicDialog.addButton().
this.footer.setHeight(this.btnContainer.offsetHeight + this.footer.getPadding('tb'));
This also fixed it in IE6 strict mode.
4 Feb 2007 2:34 PM #9Originally Posted by jacksloc
what div does 'tb' refer to? the body dialog div? the footer?
also syncBodyHeight() is nowhere in createDialog -- and I don't see it in the docs either.. can you elaborate please?
do you have a working example that I can see live?
4 Feb 2007 2:45 PM #10
syncBodyHeight is a private method in BasicDialog.js
Take a look at Element.getPadding(). Like other functions in Element, it accepts a standard set of "sides" as a string. getPadding("tb") means parse and sum the padding-top ('t') and padding-bottom ('b').
By zeek in forum Ext 1.x: Help & DiscussionReplies: 2Last Post: 5 Mar 2007, 11:05 AM
By kalebwalton in forum Ext 1.x: Help & DiscussionReplies: 6Last Post: 6 Feb 2007, 12:37 PM
By slidenv in forum Ext 1.x: Help & DiscussionReplies: 1Last Post: 5 Jan 2007, 12:01 AM
By alex1er in forum Ext 1.x: Help & DiscussionReplies: 1Last Post: 9 Nov 2006, 6:56 AM