diff --git a/app/views/users/list.html.erb b/app/views/users/list.html.erb
index 177bf7391f0..472b1ea21b6 100644
--- a/app/views/users/list.html.erb
+++ b/app/views/users/list.html.erb
@@ -1,29 +1,35 @@
Manage users
-<%= form_tag :action => 'show_if_authorized' do %>
- View user - enter exact login: <%= text_field_with_auto_complete :user, :username, {:size => 41} %>
-
-<% end %>
-
-<%= form_tag :action => 'list', :paginate_show => '1', :num_users => @per_page do %>
- <%= label_tag :letter, 'Search users: ' %><%= text_field_tag(:letter,params[:letter]) %>
-<%= select_tag(:search_by, options_for_select([['Username', 1], ['Name', 2], ['Email', 3]])) %>
-<%= submit_tag("Search") %>
-<% end %>
+<%= form_tag :action => 'list', method: :get, id: 'pagination_form' do %>
+ <%= label_tag :letter, 'Search users: ' %><%= text_field_tag(:letter, params[:letter]) %>
+ <%= select_tag(:search_by, options_for_select([['Username', 1], ['Full name', 2], ['Email', 3]])) %>
+ <%= submit_tag("Search") %>
-<%= form_tag :action => 'list', :paginate_show => '1', :num_users => @per_page do %>
- <%= label_tag :letter, 'Number of users displayed per page' %><%= text_field_tag(:letter,params[:per_page]) %>
-<%= select_tag(:search_by, options_for_select([['25 users', 1], ['50 users', 2], ['100 users', 3], ['All users', 4]])) %>
-<%= submit_tag("List") %>
+
+
+ <%= select_tag(:per_page, options_for_select(
+ [['25 users', '1'], ['50 users', '2'], ['100 users', '3'], ['All users', '4']],
+ selected: @per_page
+ ),
+ onchange: "document.getElementById('pagination_form').submit();"
+ ) %>
<% end %>
-<%= render :partial => 'links' %>
-
| E-mail on ... |
- User Name | Name | Email Address | Role | Parent | Review | Submission | Meta-review | Delete |
+
+ User Name |
+ Full Name |
+ Email Address |
+ Role |
+ Parent |
+ Review |
+ Submission |
+ Meta-review |
+ Delete |
+
<% @paginated_users.each do |user| %>
<% if (params[:show] != 'true' && !user.username(session[:ip]).include?("_hidden")) || params[:show] == 'true'%>
@@ -41,10 +47,6 @@
<% end -%>
-
-
-
-<%#= render :partial => 'shared_scripts/pages' %>
<%= will_paginate @paginated_users, :inner_window => 1, :outer_window => 1, :params => {:num_versions => @per_page}%>
@@ -59,5 +61,4 @@
<%= link_to 'New user',:role => "Student", :action => 'new' %> |
<%= link_to 'Import users', :controller=>'import_file', :action=>'start', :model => 'User', :expected_fields => 'username | full name (first [middle] last) | e-mail address' %> |
<%= link_to 'Export users', :controller=>'export_file',:action=>'start',:model=> 'User',:id=> 1 %>
-