#6559 closed task (fixed)
Tooltip: make pointer align with text
Reported by: | bill | Owned by: | bill |
---|---|---|---|
Priority: | high | Milestone: | 1.6 |
Component: | Dijit - LnF | Version: | 1.1.0 |
Keywords: | ibm | Cc: | |
Blocked By: | Blocking: |
Description
A tooltip next to text looks too high:
The center of the pointer should align w/the center of the text.
Need to be careful with this one because it could cause the top or the bottom of the tooltip to go offscreen, if the target text is near the top or the bottom of the viewport.
Also need to define the spec exactly... is the arrow vertically in the middle of the bubble? Currently it's towards the bottom.
Taken from IBM Design Leadership recommendations.
Attachments (2)
Change History (8)
Changed 13 years ago by
Attachment: | tooltip.gif added |
---|
comment:1 Changed 13 years ago by
comment:2 Changed 12 years ago by
Milestone: | 1.4 → future |
---|
The workaround to this issue is to use above/below tooltips instead.
comment:4 Changed 10 years ago by
Unlike #4997, this requires vertical middle alignment support in dijit.place, maybe support CL/CR in addition to TL/BL/TR/BR for the alignment parameter. (C == center.)
comment:5 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
(In [22995]) Always vertically center the arrow with the around node, rather than just when it doesn't point at the around node. This fixes the test failures on FF/mac which were due to special code for having the arrow 3px above the bottom of the tooltip. Refs #4997 !strict.
This also makes the arrow align better with small around nodes (fixes #6559), although I think the ideal fix would still be to vertically center the entire tooltip with the around node.
Patch from Katie (IBM, CCLA), thanks!
comment:6 Changed 10 years ago by
Milestone: | future → 1.6 |
---|
Here's how it looks after [22995], and also I think the arrow got smaller in claro:
So even for plain text (which is vertically short), the arrow looks aligned.
(In [13430]) Tooltip look and feel updates: