Defect #30287

The tooltip layout of the calendar is broken

Added by Mizuki ISHIKAWA 24 days ago. Updated 2 days ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:Calendar
Target version:4.0.1
Resolution:Fixed Affected version:4.0.0

Description

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

Expected output Actual output

actual.png (71.4 KB) Mizuki ISHIKAWA, 2018-12-28 03:05

expected.png (82.4 KB) Mizuki ISHIKAWA, 2018-12-28 03:05

Associated revisions

Revision 17821
Added by Jean-Philippe Lang 2 days ago

The tooltip layout of the calendar is broken (#30287).

Patch by Mizuki ISHIKAWA.

Revision 17822
Added by Jean-Philippe Lang 2 days ago

Merged r17821 to 4.0-stable (#30287).

History

#1 Updated by Mizuki ISHIKAWA 24 days 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;

#2 Updated by Go MAEDA 24 days ago

  • Target version set to 4.0.1

#3 Updated by Marius BALTEANU 22 days 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?

#4 Updated by Jean-Philippe Lang 2 days 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.

Also available in: Atom PDF