Opened 12 years ago

Closed 5 years ago

Last modified 5 years ago

#4284 closed feature (fixed)

Tooltip: support hideDelay

Reported by: guest Owned by: bill
Priority: low Milestone: 1.10
Component: Dijit Version: 0.9
Keywords: Tooltips hideDelay Cc:
Blocked By: Blocking:

Description

In Dojo 0.4 you could delay fading out a tooltip. Now hideDelay seems not to be supported any longer. That is a disadvantage compared to 0.4.

Attachments (1)

hideDelay.patch (9.8 KB) - added by bill 7 years ago.
implement hideDelay and allow hover over tooltip

Download all attachments as: .zip

Change History (13)

comment:1 Changed 12 years ago by bill

Milestone: 2.0

That's true, and I guess it's because we are sharing a single MasterTooltip? to print the hints anywhere on the screen. That was for performance reasons. Anyway this isn't an easy thing to fix.

comment:2 Changed 12 years ago by alex

Milestone: 2.01.3

Milestone 2.0 deleted

comment:3 Changed 11 years ago by bill

Component: GeneralDijit
Milestone: 1.3future
Owner: anonymous deleted

comment:4 Changed 7 years ago by bill

Priority: highlow

comment:5 Changed 7 years ago by dylan

Milestone: future1.9
Resolution: patchwelcome
Status: newclosed

I'm thinking this should be a wontfix after six years, but we would welcome a patch for 2.0

comment:6 Changed 7 years ago by bill

Milestone: 1.92.0
Resolution: patchwelcome
Status: closedreopened
Type: defectenhancement

I'm actually working on a patch, but does this actually benefit end users? If so, how?

Changed 7 years ago by bill

Attachment: hideDelay.patch added

implement hideDelay and allow hover over tooltip

comment:7 Changed 7 years ago by bill

Owner: set to bill
Status: reopenedassigned

The patch above will:

  1. Reimplement hideDelay timer from 0.4, fixes #4284 !strict
  2. Allow hovering over tooltip (in addition to hovering over the anchor node, which fixes problem when tooltip covers target node causing a spurious mouseleave on the target, fixes #8267 !strict.

(2) also moves towards allowing scrolling of a Tooltip to work, refs #8680.

The patch is against [31066], and passes regression. It should have some tests added though.

comment:8 Changed 7 years ago by bill

Milestone: 2.01.10

Might as well check this into SVN and then merge to github, so it's available for the 1.x and 2.x streams.

comment:9 in reply to:  7 Changed 6 years ago by freddefisk

Replying to bill:

  1. Allow hovering over tooltip...

(2) also moves towards allowing scrolling of a Tooltip to work, refs #8680.

This will be a welcome feature. Let me know if you need any help to get this into trunk.

comment:10 Changed 6 years ago by bill

freddefisk - Yes, I need automated tests before I can check it in.

comment:11 Changed 5 years ago by Bill Keese <bill@…>

Resolution: fixed
Status: assignedclosed

In f36259200942691b1ff4a7542eedf3c52a4e588c/dijit:

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

comment:12 Changed 5 years ago by bill

Summary: Tooltips: hideDelay not supported any longer?Tooltip: support hideDelay
Type: enhancementfeature
Note: See TracTickets for help on using tickets.