popover placement bug #23

Closed
MarcusJT opened this Issue Jul 3, 2012 · 9 comments

Comments

Projects
None yet
4 participants

MarcusJT commented Jul 3, 2012

The popover always appears at the right of the end of the gantt item, which is fine when the item is fully visible in the charting area but causes problems when it extends beyond the right hand side, whereupon the popover appears where the item would end if the rest of it were visible. This is particularly evident on the very long "training" item in the demo chart - scroll the area so that the item is half visible on the right, then hover over it.

I suggest that the placement of the popover (for items which extend beyond the edge of the charting area) should be clipped to the right hand side of the charting area, inside the border, i.e. so that it points to the rightmost (clipped) edge of the item.

sithik commented Jul 13, 2012

I removed popover and brought in jQuery Tooltip Plugin (http://bassistance.de/jquery-plugins/jquery-plugin-tooltip/)

Also I added a call back code to bind the tooltip on every data rendering. it working like charm.

Sounds great, but the demo at http://taitems.github.com/jQuery.Gantt/ still looks the same, please update it?

sithik commented Jul 26, 2012

Marcus, I am not a developer of this library. I just used this library and
customized in my application. You can easily add a callback and in that
callback you shall use other tooltip.

On Thu, Jul 26, 2012 at 8:59 AM, Marcus <
reply@reply.github.com

wrote:

Sounds great, but the demo at http://taitems.github.com/jQuery.Gantt/still looks the same, please update it?


Reply to this email directly or view it on GitHub:
#23 (comment)

Sithik
Catch me here Twitter http://twitter.com/abusithik,
Facebookhttp://www.facebook.com/sithikand
LinkedIn http://in.linkedin.com/in/sithik

Ah, I see. Thanks!

Owner

taitems commented Jul 30, 2012

That plugin seems pretty old. I'm trying to preserve bootstrap look and feel as it has pretty active development.

I think the fix for this is to ignore gravity (North East, South West etc) and just have it coming out the top and bound to the x axis position.

taitems was assigned Jul 30, 2012

That's a workaround, but not really a fix - a fix would implement the behaviour I describe, as one would expect.

Owner

taitems commented Nov 17, 2012

This will be resolved when this bug is resolved in the bootstrap source.

If you need it fixed immediately I would recommend downloading the persons modified code. Until then I'll leave this one open.

That sounds fine but that particular bug's been closed as a dupe - one of many - so this is the master pull request to keep an eye on instead: twitter/bootstrap#5768

Collaborator

usmonster commented Jul 23, 2013

Upstream bug seems fixed, closing this bug.

usmonster closed this Jul 23, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment