Project

General

Profile

Actions

Defect #28204

closed

Too large avatar breaks gantt when assignee is a group

Added by Go MAEDA about 6 years ago. Updated about 6 years ago.

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

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

A 128px wide avatar icon breaks gantt view when the assignee of an issue is a group and "Use Gravatar user icons" setting is enabled.

The problem is introduced in Redmine 3.3.6 and 3.4.4 by #26699. The cause is that the HTML doesn't include "width" and "height" option because image_tag in Rails 4 doesn't accept an integer value for ":size" option (Rails 5 accepts both integer and string value, so the current trunk is not affected).

Redmine 3.4 (img element doesn't have "width" and "height" attributes):

<img alt="" title="Assignee: A Team" class="gravatar icon-gravatar" default="default" src="/images/anonymous.png?1518850381">

The current trunk (img element has "width" and "height" attributes):

<img alt="" title="Assignee: A Team" class="gravatar icon-gravatar" default="default" src="/images/anonymous.png?1517405352" width="13" height="13">


Files

too-large-icon@2x.png (36.1 KB) too-large-icon@2x.png Go MAEDA, 2018-02-17 10:33
test_size_option_for_avatar_should_accept_integer_value.diff (529 Bytes) test_size_option_for_avatar_should_accept_integer_value.diff test to catch this issue Go MAEDA, 2018-02-18 00:35
size_option_for_avatar_should_accept_integer_value.diff (537 Bytes) size_option_for_avatar_should_accept_integer_value.diff fix for this issue (no need to apply this fix to the trunk) Go MAEDA, 2018-02-18 00:36

Related issues

Related to Redmine - Defect #26699: Anonymous user should have their iconClosed

Actions
Related to Redmine - Defect #28208: Anonymous icon is wrongly displayed when assignee is a groupClosedJean-Philippe Lang

Actions
Actions #1

Updated by Go MAEDA about 6 years ago

  • Related to Defect #26699: Anonymous user should have their icon added
Actions #2

Updated by Go MAEDA about 6 years ago

Actions #4

Updated by Mischa The Evil about 6 years ago

  • Target version set to 3.3.7
Actions #5

Updated by Go MAEDA about 6 years ago

  • Related to Defect #28208: Anonymous icon is wrongly displayed when assignee is a group added
Actions #6

Updated by Go MAEDA about 6 years ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the fix to 3.4-stable and 3.3-stable branch.

Mischa, thank you for reviewing the patch.

Actions

Also available in: Atom PDF