#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)
Change History (13)
comment:1 Changed 12 years ago by
Milestone: | → 2.0 |
---|
comment:3 Changed 11 years ago by
Component: | General → Dijit |
---|---|
Milestone: | 1.3 → future |
Owner: | anonymous deleted |
comment:4 Changed 7 years ago by
Priority: | high → low |
---|
comment:5 Changed 7 years ago by
Milestone: | future → 1.9 |
---|---|
Resolution: | → patchwelcome |
Status: | new → closed |
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
Milestone: | 1.9 → 2.0 |
---|---|
Resolution: | patchwelcome |
Status: | closed → reopened |
Type: | defect → enhancement |
I'm actually working on a patch, but does this actually benefit end users? If so, how?
Changed 7 years ago by
Attachment: | hideDelay.patch added |
---|
implement hideDelay and allow hover over tooltip
comment:7 follow-up: 9 Changed 7 years ago by
Owner: | set to bill |
---|---|
Status: | reopened → assigned |
The patch above will:
- Reimplement hideDelay timer from 0.4, fixes #4284 !strict
- 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
Milestone: | 2.0 → 1.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 Changed 7 years ago by
comment:10 Changed 6 years ago by
freddefisk - Yes, I need automated tests before I can check it in.
comment:11 Changed 6 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:12 Changed 6 years ago by
Summary: | Tooltips: hideDelay not supported any longer? → Tooltip: support hideDelay |
---|---|
Type: | enhancement → feature |
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.