Opened 12 years ago

Closed 12 years ago

#10155 closed defect (fixed)

HasDropDown - popUp div isn't shown second time

Reported by: Mathevet julien Owned by: Nathan Toone
Priority: high Milestone: 1.4
Component: Dijit Version: 1.4.0b
Keywords: Cc:
Blocked By: Blocking:

Description

Hi, I have a tabContainer inside a DropDownButton?. It is show only first time. I remark if I comment in openDropDown function: 

//if(this._preparedNode){
   //dojo.style(ddNode, {width: "", height: "", display: "", visibility: "hidden"});
//}else{
   dijit.popup.prepare(ddNode);
   //this._preparedNode = true;
//}

It works...

Attachments (1)

test_dropDownMenu_Show.html​ (3.4 KB) - added by Mathevet julien 12 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 12 years ago by bill

Component: General → Dijit
Owner: changed from anonymous to Nathan Toone

Nathan, what do you think? I see that code in _HasDropDown although not sure what it's for.

comment:2 Changed 12 years ago by Nathan Toone

Milestone: tbd → 1.4

I have a fix that *should* work - though without a testcase, I am unable to test if it did.

That code mainly exists for when we are specifying or limiting the widths or heights of the popup...so I'm guessing that in this case, you are not setting any of those values.

comment:3 Changed 12 years ago by Nathan Toone

Resolution: → fixed
Status: new → closed

(In [20622]) Fixes #10155 - change the logic on when we call popup.prepare

comment:4 Changed 12 years ago by Mathevet julien

Resolution: fixed
Status: closed → reopened

it seem not fixed. I have an autoWidth setted ( I don't why...). Could work if you call: {{{ var mb = dojo.marginBox(ddNode); }}} before reseting ddNode width...

Changed 12 years ago by Mathevet julien

Attachment: test_dropDownMenu_Show.html​ added

comment:5 Changed 12 years ago by Nathan Toone

(In [20627]) Refs #10155 roll back last checkin - causes more problems than it fixes

comment:6 Changed 12 years ago by Nathan Toone

Resolution: → fixed
Status: reopened → closed

(In [20628]) Fixes #10155 - prevent resetting of size if the dropdown has explicit height or width specified on it to begin with

Note: See TracTickets for help on using tickets.