Project

General

Profile

Actions

Defect #30287

closed

The tooltip layout of the calendar is broken

Added by Mizuki ISHIKAWA about 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Category:
Calendar
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

The layout of the assignee icon in the tooltip of the calendar is broken.

Expected output Actual output

Files

actual.png (71.4 KB) actual.png Mizuki ISHIKAWA, 2018-12-28 03:05
expected.png (82.4 KB) expected.png Mizuki ISHIKAWA, 2018-12-28 03:05
Actions #1

Updated by Mizuki ISHIKAWA about 5 years ago

I think that it will be fixed by adding the following CSS.

diff --git a/public/stylesheets/application.css b/public/stylesheets/application.css
index 039cc07a9d..bff156820a 100644
--- a/public/stylesheets/application.css
+++ b/public/stylesheets/application.css
@@ -962,6 +962,11 @@ p.cal.legend span {display:block;}
 .tooltip span.tip{display: none; text-align:left;}
 .tooltip span.tip a { color: #169 !important; }

+.tooltip span.tip img.gravatar {
+  float: none;
+  margin: 0;
+}
+
 div.tooltip:hover span.tip{
 display:block;
 position:absolute;

Actions #2

Updated by Go MAEDA about 5 years ago

  • Target version set to 4.0.1
Actions #3

Updated by Marius BÄ‚LTEANU about 5 years ago

What do you think if we refactor the tooltip and use a list or div elements (like in the issue page) to render the issue attributes instead of the current implementation and proposed fix?

Actions #4

Updated by Jean-Philippe Lang about 5 years ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

Patch committed, thanks.

Marius BALTEANU wrote:

What do you think if we refactor the tooltip and use a list or div elements (like in the issue page) to render the issue attributes instead of the current implementation and proposed fix?

If it simplifies the css, that would be great. Please feel free to submit a patch for next release.

Actions

Also available in: Atom PDF