Tooltip: after/before position broken in claro

[Karl Tiedt]

​http://downloads.dojotoolkit.org/release-1.7.2/dojo-release-1.7.2/dijit/themes/themeTester.html

Expand the Popups and Alerts accordion, then mouse over the before/after tooltip areas... the arrows are not connected positioned to the tooltip

[bill]

This keeps coming up, see #14701 and #14976. I already documented that tooltip position needs to be after-centered/before-centered, but for back-compat I'll add some code to convert after/before to after-centered/before-centered.

[bill]

In [28168]:

Get after/before to work for Tooltip again, fixes #15016 on trunk !strict.

[bill]

In [28169]:

Get after/before to work for Tooltip again, fixes #15016 on 1.7 branch !strict.

[Karl Tiedt]

I saw those actually, and [27718] sounds like before/after *should* work like they used to (which would be expected for backwards-compat) based on the commit message.

[Karl Tiedt]

Bill, is it safe to say that "above/below/before/after" are all deprecated and no longer officially supported? If so we might want to include mapping above/below the same way you did before/after

If so then I'll refrain from mentioning the next thing we noticed involving above-centered + above.

[bill]

Hmm, well I'm recommending above-centered/below-centered rather than above/below as I think they look nicer, but I wasn't thinking so far as deprecated. Although maybe I'll change my mind when I hear the problem you are having.

I could do the mapping like you suggested but I was a little worried someone might want the old behavior, where the left side of the tooltip is aligned with the left side of the anchor node. I think that alignment looks good (or normal?) for TooltipDialog on desktop. I know TooltipDialog is a different widget than Tooltip, but still something to think about for comparison.

[Karl Tiedt]

I didnt find a test in ThemeTester? that reproduced this but if you have a above-centered tooltip and display it, then display a above tooltip the connector is stil "centered" due to this if block located here:

​http://bugs.dojotoolkit.org/browser/dojo/dijit/trunk/Tooltip.js#L106

if you add an else { this.connectorNode.style.left = ""; } the problem goes away.

[bill]

Makes sense, thanks, I'll add that.

[bill]

In [28192]:

Fix problem where if you have a above-centered tooltip and display it, then display a above tooltip the connector is stil "centered". Refs #15016 !strict.