Opened 7 years ago

Closed 7 years ago

Last modified 7 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 7 years ago.
Wrong tooltip layout before patch
15609.patch (1.4 KB) - added by Eric Durocher 7 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 7 years ago by Eric Durocher

Attachment: tooltip_KO.PNG added

Wrong tooltip layout before patch

comment:1 Changed 7 years ago by Eric Durocher

Priority: undecidedhigh
Version: 1.7.31.8.0b1

Changed 7 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 7 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 7 years ago by Eric Durocher

Status: newassigned

comment:4 Changed 7 years ago by cjolif

Resolution: fixed
Status: assignedclosed

In [29128]:

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

comment:5 Changed 7 years ago by bill

Milestone: tbd1.8
Note: See TracTickets for help on using tickets.