Tooltip: places the tooltips wrong when window is very small.

[guest]

Test the following example. Make the window very small so that the lines in the tooltip and from the text right to the checkboxes will be broken. The tooltip will be positioned far away from the checkbox.

[bill]

I had to update your test file some (attached the new version) but I am able to reproduce.

[bill]

put in root directory, next to dojo/ and dijit/

[bill]

further simplification of test

[bill]

(In [22969]) Tooltip positioning fixes:

• Before positioning tooltip, limit tooltip's width to available width between the around node and the edge of the viewport. Reducing a tooltip's width (for tooltips with flowing text) will increase the tooltip's height, which affects vertical placement.

• Position tooltip connector arrow to line up with the around node, even when tooltip (due to a large tooltip size/small viewport size) extends both above and below the around node.

• Fix problem with tooltip placement when clicking a TextBox causes it to scroll into view simultaneously with displaying a tooltip.

Note: To determine the "natural" width of a toolip, had to remove left:50% setting for offscreen tooltips, because offscreen tooltips still get limited in width to the edge of the viewport. That CSS was apparently to work around a bug in an old browser (but no longer happens in the browsers we support).

Patch from Katie Vance (IBM, CCLA), thanks!

Fixes #4997, #4282 !strict.

Known issues: regression test failing by a few pixels on Mac FF (but working on Windows FF).

[haysmark]

Bill, the new Tooltip_placement test has a lot of redundant test case declarations. I attached a patch to make the test file more maintainable.

[bill]

Thanks, that looks like a much cleaner design (for the test), but even before [22995], I'm getting 4 failures (tested on IE6 and FF3.6/win). Not sure if those indicate a problem w/the test or with the code?

After [22995], I'm getting many test failures.

[haysmark]

Yes before [22995] I was getting a few failures because the tooltip wasn't appearing on screen for very long textboxes. Now the _abs and _ab tests systematically fail for me.

[haysmark]

Fixes #4282. Refactor Tooltip_placement to use 1/5th the LOC.

[haysmark]

I fixed those 4 failures and reattached the patch. It turned out that one of the textboxes wasn't supposed to be tested, so 1 textbox*4 groups=4 failures. Also, sometimes I noticed the robot took an extremely long time to click the above,below button; this would cause the _abs and _ab tests to fail. Restarting the browser fixed it for me.

[bill]

(In [23000]) Consolidating lots of repeated test code, patch from Mark Hays (IBM, CCLA), thanks! Fixes #4282.

[Douglas Hays]

(In [23001]) Refs #4282. Add test actions based on events to greatly speed up the test. I'm getting no failures on IE6, FF3.6, Safari 5.0.

[Douglas Hays]

(In [23015]) Refs #4282. Change test_Tooltip_placement.html to work better with small browser window sizes and with IE6, to use claro by default, and to validate against HTML5.

[Katie Vance]

Doug, I like the idea of using <br>'s. I think it may have uncovered a real bug. However, we still need the tooltips to have more content because they must span the entire width available in order to test the main test case. Making them as small as you did caused the tests to fail on a normal size window. I'm attaching a patch to make them just a bit larger. The patch also addresses a bug I found after I resized the tooltips.

[Katie Vance]

I'm also seeing the tests fail on IE8. I think my browser is running the last sets of tests so fast that they are validating the placement of the tooltip before the tooltip is placed. I will look into that tomorrow.

[Katie Vance]

place13.patch is a cumulative patch that includes patches 11 and 12. I deleted both patches for 11 & 12 for that reason.

[Katie Vance]

The new placement tests require a min browser size of 500px X 500px. Any smaller and some of the very small tooltips will be forced to overlap textboxes and fail tests because of the new nowrap behavior. Only the nowrap tests are allowed to overlap textboxes.

[Katie Vance]

This new patch runs on 400X400 min browser size.

[Katie Vance]

Cumulative patch that replaces place13

[bill]

(In [23065]) Fix from Katie so that test runs again in a large browser window (not just a very small one), plus some fixes to the code itself. Fixes #4282 !strict.

[bill]

(In [23288]) Patch from Katie to avoid spurious failure when browser maximized (FF on my machine has dojo.window.getBox().w --> 1583px), refs #4282.

[bill]

(In [23520]) Don't let bottom of tooltip connector extend past bottom of tooltip content, as it makes the tooltip look broken. Problem occurred with <14px tall aroundNodes such as points on charts.

Would be better to center the tooltip content relative to the center of the aroundNode, rather than the current behavior where their bottoms are aligned. With this checkin, for small aroundNodes (such as points on charts) the connector doesn't quite vertically align with the aroundNode.

Refs #4282, #4997, [23512], !strict.

[bill]

(In [23531]) Don't let top of tooltip connector extend past top of tooltip content, as it makes the tooltip look broken. Problem occurred with large aroundNodes such as the bubble charts in dojox/charting/tests/test_event2d.html.

Refs #4282, #4997, [23512], !strict.

[bill]

In [26937]:

Rewrite Tooltip-placement.html, removing unnecessary dependency on dijit.form.ValidationTextBox? etc. Calling dijit.Tooltip.show() should be sufficient for testing tooltip placement, and will hopefully have less intermittent failures than the old test. Refs #4282 !strict.

[bill]

In [26939]:

Remove code from [23065] to "Adjust width for tooltips that have a really long word or a nowrap setting". It doesn't work, as shown by "test14" failure in Tooltip-placement.html. It will make any tooltip with a sufficient amount of text expand to fill the width of the viewport, even if the tooltip content has neither large words nor a nowrap setting. Refs #4282 !strict.

[Bill Keese <[email protected]…>]

In ff4671ee94323a629ecc09d4b6fffdecb2a72fee/dijit:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled?