Commit e72b9ffe authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Merge branch 'fix_tooltips' into 'master'

Fix tooltip display in list views

After the recent UI changes the tooltips in a list view rendered behind the next object. This resets the tooltip's container to the body, rather than the individual `issue-info` div. In particular, this change will be applied to *all* tooltips in GitLab. This should take care of any other unidentified issues like this and should not have any negative effects that I can think of. This fixes the bug I noted in !1255.

**Current:**

![Screen_Shot_2015-09-14_at_8.45.34_AM](https://gitlab.com/dblessing/gitlab-ce/uploads/5fc6343f2e440dd10d01dfa595835e6a/Screen_Shot_2015-09-14_at_8.45.34_AM.png)

**As proposed:**

![Screen_Shot_2015-09-14_at_8.44.35_AM](https://gitlab.com/dblessing/gitlab-ce/uploads/7ffce7ce7523514bf7bb43d6380eb1d1/Screen_Shot_2015-09-14_at_8.44.35_AM.png)

See merge request !1288
parents d3f5df2b 539f30dd
......@@ -201,7 +201,7 @@ module ApplicationHelper
class: "#{html_class} js-timeago",
datetime: time.getutc.iso8601,
title: time.in_time_zone.stamp('Aug 21, 2011 9:23pm'),
data: { toggle: 'tooltip', placement: placement }
data: { toggle: 'tooltip', placement: placement, container: 'body' }
element += javascript_tag "$('.js-timeago').timeago()" unless skip_js
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment