Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#15609 closed defect (fixed)

dojox/mobile/Tooltip: "before" and "after" positions do not work any more

Reported by: Eric Durocher Owned by: Eric Durocher
Priority: high Milestone: 1.8
Component: DojoX Mobile Version: 1.8.0b1
Keywords: Cc:
Blocked By: Blocking:

Description

Calling tooltip.show(node, [ "before", "after" ]) will result in a wrong layout of the tooltip. To reproduce: open dojox/mobile/tests/test_Tooltip.html and click the "right" or "left" input elements on each side of the view.

Attachments (2)

tooltip_KO.PNG​ (2.7 KB) - added by Eric Durocher 10 years ago.
Wrong tooltip layout before patch
15609.patch​ (1.4 KB) - added by Eric Durocher 10 years ago.
Patch for broken mobile tooltip layout with position=before/after - Eric Durocher (IBM, CCLA)

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by Eric Durocher

Attachment: tooltip_KO.PNG​ added

Wrong tooltip layout before patch

comment:1 Changed 10 years ago by Eric Durocher

Priority: undecided → high
Version: 1.7.3 → 1.8.0b1

Changed 10 years ago by Eric Durocher

Attachment: 15609.patch​ added

Patch for broken mobile tooltip layout with position=before/after - Eric Durocher (IBM, CCLA)

comment:2 Changed 10 years ago by Eric Durocher

The patch replaces any "before" and "after" positions by "before-centered" and "after-centered", which are the valid values, for compatibility with existing code (similar to what is done in dijit/Tooltip.js). The API doc is also updated to reflect the new valid values.

comment:3 Changed 10 years ago by Eric Durocher

Status: new → assigned

comment:4 Changed 10 years ago by cjolif

Resolution: → fixed
Status: assigned → closed

In [29128]:

fixes #15609. Fixes tooltip issue create by a Dijit incompatibility. Thanks Eric Durocher (IBM, CCLA).

comment:5 Changed 10 years ago by bill

Milestone: tbd → 1.8
Note: See TracTickets for help on using tickets.