Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#5670 closed defect (fixed)

TooltipDialog: can't contain layout widgets

Reported by: bill Owned by: bill
Priority: high Milestone: 1.1
Component: Dijit Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

Split off from #4169:

Since layout widgets can't render inside a hidden div (at least on some browsers), tabs etc. don't work correctly inside of a TooltipDialog.

Change History (4)

comment:1 Changed 12 years ago by bill

Resolution: fixed
Status: newclosed

(In [12162]) Fixes #5670: TooltipDialog?: can't contain layout widgets Modifies TooltipDialog? to initialize in a visibility:hidden state rather than display:none, in order for possible child layout widgets to initialize correctly. Since TooltipDialog? is not a layout widget itself, note that this means that the child layout widgets must have a size specified (rather than specifying a size on the TooltipDialog? itself)

comment:2 Changed 11 years ago by bill

(In [15986]) Fix for ContentPane?-based templated widgets that contain nested layout widgets, and add test case.

For templated widgets, in order to check if there was original content, we need to check the contents of srcNodeRef before they are copied to containerNode. (Alternately could just check if anything is in containerNode.)

Fixes #5669, #5670, #5671. !strict

comment:3 Changed 11 years ago by bill

Description: modified (diff)

Note that as of [15986], [12162] is no longer necessary. OTOH I guess it isn't hurting anything.

comment:4 Changed 11 years ago by bill

(In [16186]) Fix comment, refs #5670 !strict.

Note: See TracTickets for help on using tickets.