atlascoder / tracker_hider

A plugin for redmine
https://www.redmine.org/plugins/tracker_hider
6 stars 2 forks source link

suggestions #6

Open quanvn opened 8 years ago

quanvn commented 8 years ago

For me, it is easier to find role and user(member) if they are sorted by name so here are my to suggestions. In file /views/_tracker_hider_settings.html.erb

  1. Line 5, replace original line <% Role.all.each do |role| %> with <% Role.all.each do |role| %> will sort role by name
  2. replace line 9-13 with the following <%- memberlist = [] @project.members.each do |member| if member.user.is_a?(User) then memberlist << {:id=> member.user.id, :name => member.user.name}
    end end memberlist = memberlist.sort_by {|e| e[:name]} if memberlist.any?
    -%> <% memberlist.each do |member| %>
        <% end %>

    will sort the user list by name Sorry I do not how to use git to to put these changes into a pull request. I am referring to master branch since global_role branch does not work for me

atlascoder commented 8 years ago

Ok, thanks i'll fix it!