_Widgets have no way to reliably determine whether they are visible

[Remoun Metyas]

Perhaps make _onShow and onHide set a flag (patch attached).

This may not work seamlessly with existing widgets, as some don't call this.inherited when they ovverride onHide/_onShow.

[Remoun Metyas]

Add a _showing flag to _Widgets

[bill]

Hmm, StackContainer children already have a "selected" attribute they can check, and TitlePane and Dialog have an "open" attribute.

I guess this is for TooltipDialog, Calendar, ColorPalette, etc... widgets that just happen to be used as drop downs. I suppose a "showing" flag makes sense although it makes me nervous since any random displayed widget (for example, a TabContainer) will have showing=false (or technically, showing is undefined), rather than showing=true.

[Remoun Metyas]

I agree that showing being undefined (and having to check showing !== true) is awkward. I had initially used _hidden, but found that onHide is never called for tabs that are never selected. Checking .selected and .open is awkward, too. I can't, however, think of a better alternative.

Relatedly, there's a TODO comment in dijit._base.popup: "in 2.0 standardize onShow() (used by StackContainer?) and onOpen() (used here)".

[dylan]

This would unfortunately need to be revisited given that the patch has sat around for 5 years. If there's still interest in this, please create a new pull request and we'll consider for 1.12. Closing as patchwelcome for now.

[dylan]

Given that no one has shown interest in creating a patch in the past 4+ years, I'm closing this as patchwelcome.