Feature #26409: Show assignee on gantt - Redmine

Custom queries

Documentation issues
Open defects
Open features
Patch queue
Plugin issues
Translation patches

Actions

Copy link

Feature #26409

closed

Show assignee on gantt

Added by Go MAEDA almost 7 years ago. Updated almost 5 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

Gantt

Target version:

Start date:

Due date:

% Done:

Estimated time:

Resolution:

Duplicate

Description

Needless to say, assignee is important information to manage projects. It would be great if assignee is displayed on gantt.

I have found some similar requests on the forum:

Can I display the assignee name on Gantt chart?
Show assignee on chart in better gantt plugin
Replace issue status bij issue assignee in gantt chart

Files

Download all files

show_assignee_on_gantt.patch (14 KB) show_assignee_on_gantt.patch		Mizuki ISHIKAWA, 2017-07-28 03:12
assignee_on_gantt.png (248 KB) assignee_on_gantt.png		Mizuki ISHIKAWA, 2017-07-28 03:12
show_assignee_on_gantt.patch_v2.patch (14.2 KB) show_assignee_on_gantt.patch_v2.patch		Mizuki ISHIKAWA, 2017-08-17 10:27
show_assignee_on_gantt_v3.patch (14.4 KB) show_assignee_on_gantt_v3.patch		Mizuki ISHIKAWA, 2017-11-27 03:25

Related issues

Related to Redmine - Feature #27672: Show selected columns in gantt chart

Closed

Go MAEDA

Actions

Blocked by Redmine - Feature #20481: Gantt: right and left resizable panel

Closed

Go MAEDA

Actions

Issue # Delay: days Cancel

History
Notes
Property changes

Actions

Copy link

Updated by Yuuki NARA almost 7 years ago

+1
I think so, too

Actions

Copy link

Updated by Mizuki ISHIKAWA almost 7 years ago

File show_assignee_on_gantt.patch show_assignee_on_gantt.patch added
File assignee_on_gantt.png assignee_on_gantt.png added

I wrote a patch to implement this feature.
You can switch the display like Progress line.

This patch is made on the premise that a patch of Feature #20481 (Gantt: left and right resizable panels) is applied.
Before applying this patch, apply the patch of Feature #20481 .

Actions

Copy link

Updated by Go MAEDA almost 7 years ago

Blocked by Feature #20481: Gantt: right and left resizable panel added

Actions

Copy link

Updated by Go MAEDA almost 7 years ago

Target version set to 4.1.0

I think this is a must-have feature for most gantt users. Setting target version to 4.1.0.

Actions

Copy link

Updated by Marius BĂLTEANU over 6 years ago

I've tested the patch and I've two observations:
1. the patch doesn't apply cleanly on the current trunk (r16948).
2. it'll be nice to have also gravatar shown for the assignee.

Actions

Copy link

Updated by Go MAEDA over 6 years ago

Marius BALTEANU wrote:

1. the patch doesn't apply cleanly on the current trunk (r16948).

The patch depends on another patch #20481. Could you try again after applying it?

2. it'll be nice to have also gravatar shown for the assignee.

I agree with you. Gravatar icon should be shown for consistency.

Actions

Copy link

Updated by Marius BĂLTEANU over 6 years ago

Go MAEDA wrote:

Marius BALTEANU wrote:

1. the patch doesn't apply cleanly on the current trunk (r16948).

The patch depends on another patch #20481. Could you try again after applying it?

Sorry for the mistake, it seems that my local repository didn't have the latest changes from the trunk. Now the second patch applies cleanly.

Actions

Copy link

Updated by Mizuki ISHIKAWA over 6 years ago

File show_assignee_on_gantt.patch_v2.patch show_assignee_on_gantt.patch_v2.patch added

Marius BALTEANU wrote:

…
2. it'll be nice to have also gravatar shown for the assignee.

Thank you for your advice.

I changed patch as follows:

--- a/lib/redmine/helpers/gantt.rb
+++ b/lib/redmine/helpers/gantt.rb
@@ -336,7 +336,10 @@ module Redmine
       def assigned_to_name(issue, options)
         if issue.is_a?(Issue) && options[:format] == :html && issue.assigned_to.present?
           style = "position: absolute;top: #{options[:top]}px;" 
-          assigned_to_name = view.content_tag(:div, view.content_tag(:span, issue.try(:assigned_to).try(:name)), :style => style, :class => 'issue-assigned-name')
+          content = view.avatar(issue.assigned_to,
+                                 :class => 'gravatar icon-gravatar',
+                                 :size => 10) + view.link_to_user(issue.assigned_to)
+          assigned_to_name = view.content_tag(:div, content.html_safe, :style => style, :class => 'issue-assigned-name')
           @assigned_to_names << assigned_to_name
           assigned_to_name
         end

Actions

Copy link

Updated by Mizuki ISHIKAWA over 6 years ago

File show_assignee_on_gantt_v3.patch show_assignee_on_gantt_v3.patch added

I fixed the patch.

Compatible the latest trunk.
Add styles to gravatar like #26410

Actions

Copy link

#10

Updated by Mischa The Evil over 6 years ago

I don't want to ruin anyone's party, but I'd like to drop some thoughts (concerns) I have about this feature and the proposed, patched implementation:

what's the actual significance of the assignee over other attributes (priority for example, which is in my opinion evenly important as assignee in gantt context)? From this follows that depending on the use, one might have need for every single attribute and/or even custom values, which simply cannot all be implemented
for the current need to see an issue's assignee, we already have it in the pop-up by hovering over the item bar...
I think that the use of the option checkboxes like in the patch feels weird as the other two checkboxes switch (JS) functionality (lines) onto the gantt chart bars, while this checkbox switches functionality regarding gantt chart bar attribute columns

I'd like to have some more feedback on this.

Actions

Copy link

#11

Updated by Mizuki ISHIKAWA over 6 years ago

Mischa The Evil wrote:

what's the actual significance of the assignee over other attributes (priority for example, which is in my opinion evenly important as assignee in gantt context)? From this follows that depending on the use, one might have need for every single attribute and/or even custom values, which simply cannot all be implemented

Thank you for your feedback.
As you say, there are other necessary information besides the Assignee.
I think that this issue can be closed if the #27672 feature is realized.
What do you think?

I am writing a patch that realizes the feature of #27672 based on the patch of this issue now.

Actions

Copy link

#12