#14650 closed defect (fixed)
Charting Tooltips on plot2d/Bar are incorrectly placed
Reported by: | cjolif | Owned by: | cjolif |
---|---|---|---|
Priority: | high | Milestone: | 1.7.2 |
Component: | Charting | Version: | 1.7.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description (last modified by )
Charting Tooltips on plot2d/Bar are incorrectly placed
This is already fixed in trunk/1.8 however we might want to fix for 1.7.2 as well.
See #27444.
Attachments (1)
Change History (11)
comment:1 Changed 9 years ago by
comment:2 Changed 9 years ago by
Status: | new → assigned |
---|
comment:4 Changed 9 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
This fix introduces a regression. See attached test case. It seems the tooltip on the last bar it no anymore correct after the fix.
Changed 9 years ago by
Attachment: | test_14650.html added |
---|
comment:5 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Actually this is not a "real" regression, indeed the behavior was already this one in 1.6. So we are back to original behavior. It is just that this behavior is not necessarily very intuitive. However the actual positioning work is done by dijit/place not charting so there's not much we can do here.
comment:6 Changed 9 years ago by
Description: | modified (diff) |
---|
Well, you could just add a margin-top: 10px
to the style of your chart. In the test file there isn't enough room for a tooltip at the top, which is why it's putting the tooltip at the bottom. I agree it looks bad but I'd consider it a problem with the test file.
Alternately in dojox/charting/axis2d/Tooltip, could modify the logic in process to just use ["above"]
where it currently uses ["above", "below"]
.
comment:7 Changed 9 years ago by
Thanks Bill for looking at this. I agree this is not really a problem. What was questioning me was that using aroundNode.height (deprecated) vs aroundNode.h I get a different (nicer) behavior.
comment:8 Changed 9 years ago by
The same problem still exists in dojo 1.7.2
To reproduce the error (firefox 11, chrome, IE9, ...):
1- download :http://download.dojotoolkit.org/release-1.7.2/dojo-release-1.7.2-src.tar.gz 2- upload to server 3-go to dojos/dojo/charting/tests/test_event2d.html
And you can see the problem.
comment:9 Changed 9 years ago by
Ok, I reproduce, actually it is a slightly different issue. This is #15016 which is fixed in 1.7.3. Sorry for all this tooltip issues. If download a nightly (http://archive.dojotoolkit.org/nightly/) you should be able to confirm this is working now. If it is not please let me know.
I have a similar issue in ClusteredBars?
demo here
http://jsbin.com/utesoz/2/edit#source