Defect #30287

The tooltip layout of the calendar is broken

Added by Mizuki ISHIKAWA about 1 year ago. Updated about 1 year 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 about 1 year ago

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

Patch by Mizuki ISHIKAWA.

Revision 17822
Added by Jean-Philippe Lang about 1 year ago

Merged r17821 to 4.0-stable (#30287).

History

#1 Updated by Mizuki ISHIKAWA about 1 year 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 about 1 year ago

  • Target version set to 4.0.1

#3 Updated by Marius BALTEANU about 1 year 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 about 1 year 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