https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292018-04-10T11:03:40ZRedmineRedmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=844822018-04-10T11:03:40ZBernhard Rohloff
<ul></ul><p>Marius BALTEANU wrote:</p>
<blockquote>
<p>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).</p>
</blockquote>
<p>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.</p>
<blockquote>
<p>...<br />A first improvement that is quite small and easy to add is the issue assignee gravatar.</p>
</blockquote>
<p>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.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=846102018-04-23T19:45:39ZMarius BĂLTEANU
<ul></ul><p>Bernhard Rohloff wrote:</p>
<blockquote>
<p>... 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.</p>
</blockquote>
<p>Yes, I've something in mind, but I want to make a screen to discuss on it.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=846132018-04-24T04:04:52ZAkipii Oga
<ul></ul><p>+1<br />I think that the feature of the roadmap should be improved further.<br />Because the feature of the roadmap is directly linked to the operation of agile development.<br />I think that it is better for Redmine to get closer to agile development.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=846142018-04-24T04:16:27ZBernhard Rohloff
<ul></ul><p>Marius BALTEANU wrote:</p>
<blockquote>
<p>Yes, I've something in mind, but I want to make a screen to discuss on it.</p>
</blockquote>
<p>Cool! I'm looking forward to see the results. :-)</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=898642019-02-07T06:22:52ZGo MAEDA
<ul><li><strong>File</strong> <a href="/attachments/22371">gantt-icons.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/22371/gantt-icons.png">gantt-icons.png</a> added</li></ul><p>I think it is better to add an icon to the left side of the tracker name because gantt already shows gravatar icons like that.</p>
<p>Here is a screenshot of gantt. tiket.png is displayed when gravatar is disabled or no one is assigned to the issue.</p>
<p><img src="https://www.redmine.org/attachments/download/22371/gantt-icons.png" alt="" /></p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=911832019-04-10T01:36:26ZMizuki ISHIKAWA
<ul></ul><p>+1</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=915532019-05-05T16:37:34ZTakenori TAKAKItakenory@gmail.com
<ul><li><strong>File</strong> <a href="/attachments/22964">28510.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/22964/28510.patch">28510.patch</a> added</li></ul><p>+1<br />As Marius says, adding Gravatar will be a simple and useful UI improvement. I have always felt that I could confirm assigne on the version page.<br />As for placement, I agree with Mr. Maeda's suggestion.<br />I made a patch to display the avatar on the left side of the tracker name, and attache it.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=915562019-05-06T01:42:21ZGo MAEDA
<ul><li><strong>File</strong> <a href="/attachments/22966">version-gravatar-enabled@2x.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/22966/version-gravatar-enabled@2x.png">version-gravatar-enabled@2x.png</a> added</li><li><strong>File</strong> <a href="/attachments/22967">version-gravatar-disabled@2x.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/22967/version-gravatar-disabled@2x.png">version-gravatar-disabled@2x.png</a> added</li><li><strong>Assignee</strong> set to <i>Marius BĂLTEANU</i></li><li><strong>Target version</strong> set to <i>Candidate for next major release</i></li></ul><p>Takenori TAKAKI wrote:</p>
<blockquote>
<p>I made a patch to display the avatar on the left side of the tracker name, and attache it.</p>
</blockquote>
<p>Looks nice. Thanks.</p>
<p>Marius, what do you think about this patch?</p>
<p><strong>[Gravatar enabled]</strong><br />No avagar icon for issues without an assignee (Issue 1 and 12).<br /><img src="https://www.redmine.org/attachments/download/22966/version-gravatar-enabled@2x.png" srcset="https://www.redmine.org/attachments/download/22966/version-gravatar-enabled@2x.png 2x" style="width: 400px;border: 1px solid #ccc;" alt="" /></p>
<p><strong>[Gravatar disabled]</strong><br /><img src="https://www.redmine.org/attachments/download/22967/version-gravatar-disabled@2x.png" srcset="https://www.redmine.org/attachments/download/22967/version-gravatar-disabled@2x.png 2x" style="width: 400px;border: 1px solid #ccc;" alt="" /></p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=915882019-05-06T20:47:45ZMarius BĂLTEANU
<ul><li><strong>File</strong> <a href="/attachments/22988">assignee_first.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/22988/assignee_first.png">assignee_first.png</a> added</li></ul><p>I think we should take more feedback from users because displaying the ticket image ('icon-issue') to all unassigned issues is not a relevant information from my point of view. I already know that inside a version, all entries are issues. In gantt, it makes a little bit more sense because there the content is mixed (projects, versions, issues). The same is in the activity page where the content is mixed and the icons are used to make a visual difference.</p>
<p>Also, thinking at the near future and gantt, when the users will have the possibility to select the columns (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Show selected columns in gantt chart (Closed)" href="https://www.redmine.org/issues/27672">#27672</a>), maybe it'll be better to remove the avatar from issue's subject.</p>
<p>If you really want to show the assignee first, I think I prefer to not have any icon for unassigned issues:<br /><img src="https://www.redmine.org/attachments/download/22988/assignee_first.png" alt="" /></p>
<p>But again, this is just my opinion according to my tastes and how I understand that ticket image.</p>
<p>Regarding the proposed patch, I think we can slightly improve the code.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=915972019-05-07T05:33:14ZGo MAEDA
<ul></ul><p>Marius BALTEANU wrote:</p>
<blockquote>
<p>If you really want to show the assignee first, I think I prefer to not have any icon for unassigned issues:<br /><img src="https://www.redmine.org/attachments/download/22988/assignee_first.png" alt="" /></p>
</blockquote>
<p>The design is also OK for me. I agree with your opinion that ticket icons don't provide any useful information in this context.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=918352019-05-18T13:41:20ZMarius BĂLTEANU
<ul><li><strong>File</strong> <a href="/attachments/23113">0001-Show-assignee-avatar-in-Roadmap-and-Version.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/23113/0001-Show-assignee-avatar-in-Roadmap-and-Version.patch">0001-Show-assignee-avatar-in-Roadmap-and-Version.patch</a> added</li><li><strong>Assignee</strong> changed from <i>Marius BĂLTEANU</i> to <i>Go MAEDA</i></li></ul><p>Here is the updated patch that can be applied after <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Patch: Small refactorization of avatar methods (Closed)" href="https://www.redmine.org/issues/31391">#31391</a> is committed.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=919122019-05-20T15:11:07ZGo MAEDA
<ul><li><strong>File</strong> <a href="/attachments/23124">uneven-issues@2x.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/23124/uneven-issues@2x.png">uneven-issues@2x.png</a> added</li></ul><p>Marius BALTEANU wrote:</p>
<blockquote>
<p>Here is the updated patch that can be applied after <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Patch: Small refactorization of avatar methods (Closed)" href="https://www.redmine.org/issues/31391">#31391</a> is committed.</p>
</blockquote>
<p>The patch works fine but I prefer the layout of <a class="attachment" href="https://www.redmine.org/attachments/22988">assignee_first.png</a> is better the layout of the latest patch because the start position of tracker names is not aligned with the patch applied.</p>
<p><img src="https://www.redmine.org/attachments/download/23124/uneven-issues@2x.png" srcset="https://www.redmine.org/attachments/download/23124/uneven-issues@2x.png 2x" style="width: 353px;border: 1px solid #ccc;" alt="" /></p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=919302019-05-21T07:15:34ZTakenori TAKAKItakenory@gmail.com
<ul></ul><p>To specify the style of the list element, <br />How about making the following small changes to the Mariu's patch?</p>
<pre><code class="diff syntaxhl"><span class="gh">diff --git a/app/views/versions/index.html.erb b/app/views/versions/index.html.erb
index 856ee69ea..4bd61b97b 100644
</span><span class="gd">--- a/app/views/versions/index.html.erb
</span><span class="gi">+++ b/app/views/versions/index.html.erb
</span><span class="p">@@ -30,7 +30,7 @@</span>
<% issues.each do |issue| -%>
<tr class="issue hascontextmenu">
<td class="checkbox"><%= check_box_tag 'ids[]', issue.id, false, :id => nil %></td>
<span class="gd">- <td class="subject"><%= assignee_avatar(issue.assigned_to, :size => 16, :class => 'icon-gravatar').html_safe + link_to_issue(issue, :project => (@project != issue.project)) %></td>
</span><span class="gi">+ <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>
</span> <td class="buttons"><%= link_to_context_menu %></td>
</tr>
<% end -%>
<span class="gh">diff --git a/app/views/versions/show.html.erb b/app/views/versions/show.html.erb
index 6092c9792..65b82fc1b 100644
</span><span class="gd">--- a/app/views/versions/show.html.erb
</span><span class="gi">+++ b/app/views/versions/show.html.erb
</span><span class="p">@@ -44,7 +44,7 @@</span>
<%- @issues.each do |issue| -%>
<tr class="issue hascontextmenu">
<td class="checkbox"><%= check_box_tag 'ids[]', issue.id, false, :id => nil %></td>
<span class="gd">- <td class="subject"><%= assignee_avatar(issue.assigned_to, :size => 16, :class => 'icon-gravatar').html_safe + link_to_issue(issue, :project => (@project != issue.project)) %></td>
</span><span class="gi">+ <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>
</span> <td class="buttons"><%= link_to_context_menu %></td>
</tr>
<% end %>
<span class="gh">diff --git a/public/stylesheets/application.css b/public/stylesheets/application.css
index 03fd81441..0e0115d43 100644
</span><span class="gd">--- a/public/stylesheets/application.css
</span><span class="gi">+++ b/public/stylesheets/application.css
</span><span class="p">@@ -587,6 +587,8 @@</span> 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%;}
<span class="gi">+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; }
</span>
div#version-summary { float:right; width:28%; margin-left: 16px; margin-bottom: 16px; background-color: #fff; }
div#version-summary fieldset { margin-bottom: 1em; }
</code></pre> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=919422019-05-21T16:15:13ZMarius BĂLTEANU
<ul><li><strong>File</strong> <a href="/attachments/23129">0001-Show-assignee-avatar-in-Roadmap-and-Version.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/23129/0001-Show-assignee-avatar-in-Roadmap-and-Version.patch">0001-Show-assignee-avatar-in-Roadmap-and-Version.patch</a> added</li><li><strong>File</strong> <a href="/attachments/23128">0002-Add-issue-css_classes-to-issue-rows.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/23128/0002-Add-issue-css_classes-to-issue-rows.patch">0002-Add-issue-css_classes-to-issue-rows.patch</a> added</li></ul><p>Thanks Takenori for the patch.</p>
<p>What do you think if we render the assignee in its own cell?</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=919562019-05-22T05:41:36ZTakenori TAKAKItakenory@gmail.com
<ul></ul><p>Thanks Marius for new suggestion.</p>
<p>Marius BALTEANU wrote:</p>
<blockquote>
<p>What do you think if we render the assignee in its own cell?</p>
</blockquote>
<p>I think that the code is cleaner and easier to improve, If we render the assignee in its own cell.<br />Your patches( <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Patch: Show issue assignee gravatar in roadmap and version page (Closed)" href="https://www.redmine.org/issues/28510#note-14">#28510#note-14</a> ) works fine in my environment.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=919572019-05-22T06:09:36ZGo MAEDA
<ul><li><strong>Assignee</strong> changed from <i>Go MAEDA</i> to <i>Marius BĂLTEANU</i></li></ul><p>The patch in <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Patch: Show issue assignee gravatar in roadmap and version page (Closed)" href="https://www.redmine.org/issues/28510#note-14">#28510#note-14</a> looks nice. Thanks!</p>
<p>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?</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=919582019-05-22T06:28:35ZMarius BĂLTEANU
<ul></ul><p>Go MAEDA wrote:</p>
<blockquote>
<p>The patch in <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Patch: Show issue assignee gravatar in roadmap and version page (Closed)" href="https://www.redmine.org/issues/28510#note-14">#28510#note-14</a> looks nice. Thanks!</p>
<p>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?</p>
</blockquote>
<p>Agree. Let commit only the first patch which will fix also the error reported in <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Patch: Small refactorization of avatar methods (Closed)" href="https://www.redmine.org/issues/31391">#31391</a> and I'll open a new issue later with the second patch.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=919652019-05-22T08:33:44ZGo MAEDA
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Resolved</i></li><li><strong>Target version</strong> changed from <i>Candidate for next major release</i> to <i>4.1.0</i></li></ul><p>Committed the patch. Thank you all for working on this feature.</p>
<p>Marius, could you open a new issue for the second patch?</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=919782019-05-22T16:22:38ZGo MAEDA
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li><li><strong>Assignee</strong> changed from <i>Marius BĂLTEANU</i> to <i>Go MAEDA</i></li></ul><p>Go MAEDA wrote:</p>
<blockquote>
<p>Marius, could you open a new issue for the second patch?</p>
</blockquote>
<p>I have confirmed <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Patch: Add issue css classes to issue rows in Roadmap and Version pages (Closed)" href="https://www.redmine.org/issues/31424">#31424</a>. Thanks.</p> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=928172019-07-18T15:07:28ZMarius BĂLTEANU
<ul><li><strong>Has duplicate</strong> <i><a class="issue tracker-3 status-8 priority-4 priority-default" href="/issues/31752">Patch #31752</a>: Show assigned people when displaying tickets which are related to a specific versions</i> added</li></ul> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=933492019-08-28T02:47:23ZGo MAEDA
<ul><li><strong>Related to</strong> <i><a class="issue tracker-3 status-5 priority-4 priority-default closed" href="/issues/31964">Patch #31964</a>: VersionsControllerTest#test_show randomly fails</i> added</li></ul> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=933882019-08-30T00:20:46ZGo MAEDA
<ul><li><strong>Category</strong> changed from <i>UI</i> to <i>Roadmap</i></li></ul> Redmine - Patch #28510: Show issue assignee gravatar in roadmap and version pagehttps://www.redmine.org/issues/28510?journal_id=951662019-12-06T09:42:36ZGo MAEDA
<ul><li><strong>Related to</strong> <i><a class="issue tracker-2 status-1 priority-4 priority-default" href="/issues/31704">Feature #31704</a>: Show Gravatar icon in issues list.</i> added</li></ul>