Redmine

Marius BALTEANU wrote:

I'm trying to make the roadmap and version page more useful for the users. I believe that this module should be one of the most important modules from Redmine, but currently it does provide only some basic features or information (from my/our users point of view).

I totally agree with you that the Roadmap module is one of Redmine's most serious modules and there's a lot of 'untouched' potential in it. If you have any kind of concept/vision for it, it would be great if we could have a glance/discussion on it. Maybe in the forums or somewhere else.

...
A first improvement that is quite small and easy to add is the issue assignee gravatar.

I think it's a good improvement for the issues list. I personally would prefer to have the icon on the left side besides the issues name. On widescreen monitors the distance between the both sides of the table becomes rather huge and there's much empty space in between. Often this makes it hard for me to correlate the content on both sides with each other.

Bernhard Rohloff wrote:

... If you have any kind of concept/vision for it, it would be great if we could have a glance/discussion on it. Maybe in the forums or somewhere else.

Yes, I've something in mind, but I want to make a screen to discuss on it.

+1
I think that the feature of the roadmap should be improved further.
Because the feature of the roadmap is directly linked to the operation of agile development.
I think that it is better for Redmine to get closer to agile development.

Marius BALTEANU wrote:

Yes, I've something in mind, but I want to make a screen to discuss on it.

Cool! I'm looking forward to see the results. :-)

+1

Marius BALTEANU wrote:

If you really want to show the assignee first, I think I prefer to not have any icon for unassigned issues:

The design is also OK for me. I agree with your opinion that ticket icons don't provide any useful information in this context.

To specify the style of the list element,
How about making the following small changes to the Mariu's patch?

diff --git a/app/views/versions/index.html.erb b/app/views/versions/index.html.erb
index 856ee69ea..4bd61b97b 100644
--- a/app/views/versions/index.html.erb
+++ b/app/views/versions/index.html.erb
@@ -30,7 +30,7 @@
         <% issues.each do |issue| -%>
           <tr class="issue hascontextmenu">
             <td class="checkbox"><%= check_box_tag 'ids[]', issue.id, false, :id => nil %></td>
-            <td class="subject"><%= assignee_avatar(issue.assigned_to, :size => 16, :class => 'icon-gravatar').html_safe + link_to_issue(issue, :project => (@project != issue.project)) %></td>
+            <td class="subject"><%= assignee_avatar(issue.assigned_to, :size => 16, :class => 'icon-gravatar').html_safe + content_tag('span', link_to_issue(issue, :project => (@project != issue.project))) %></td>
             <td class="buttons"><%= link_to_context_menu %></td>
           </tr>
         <% end -%>
diff --git a/app/views/versions/show.html.erb b/app/views/versions/show.html.erb
index 6092c9792..65b82fc1b 100644
--- a/app/views/versions/show.html.erb
+++ b/app/views/versions/show.html.erb
@@ -44,7 +44,7 @@
   <%- @issues.each do |issue| -%>
     <tr class="issue hascontextmenu">
       <td class="checkbox"><%= check_box_tag 'ids[]', issue.id, false, :id => nil %></td>
-      <td class="subject"><%= assignee_avatar(issue.assigned_to, :size => 16, :class => 'icon-gravatar').html_safe + link_to_issue(issue, :project => (@project != issue.project)) %></td>
+      <td class="subject"><%= assignee_avatar(issue.assigned_to, :size => 16, :class => 'icon-gravatar').html_safe + content_tag('span', link_to_issue(issue, :project => (@project != issue.project))) %></td>
       <td class="buttons"><%= link_to_context_menu %></td>
     </tr>
   <% end %>
diff --git a/public/stylesheets/application.css b/public/stylesheets/application.css
index 03fd81441..0e0115d43 100644
--- a/public/stylesheets/application.css
+++ b/public/stylesheets/application.css
@@ -587,6 +587,8 @@ div#roadmap .wiki h1 { font-size: 120%; }
 div#roadmap .wiki h2 { font-size: 110%; }
 div#roadmap h2, div#roadmap h3 { display: inline; padding-right: 0;}
 body.controller-versions.action-show div#roadmap .related-issues {width:70%;}
+body.avatars-on div#roadmap .related-issues td.subject img { position: absolute; }
+body.avatars-on div#roadmap .related-issues td.subject span { padding-left: 20px; display: inline-block; }

 div#version-summary { float:right; width:28%; margin-left: 16px; margin-bottom: 16px; background-color: #fff; }
 div#version-summary fieldset { margin-bottom: 1em; }

Thanks Marius for new suggestion.

Marius BALTEANU wrote:

What do you think if we render the assignee in its own cell?

I think that the code is cleaner and easier to improve, If we render the assignee in its own cell.
Your patches( #28510#note-14 ) works fine in my environment.

Go MAEDA wrote:

The patch in #28510#note-14 looks nice. Thanks!

But 0002-Add-issue-css_classes-to-issue-rows.patch seems to implement another feature different from this issue. Gravatars are shown correctly, even without the patch. I think it is better to open a new issue to add CSS classes. What do you think?

Agree. Let commit only the first patch which will fix also the error reported in #31391 and I'll open a new issue later with the second patch.