Move current user to the top of the list in author/assignee filter (!505) · Merge requests · GitLab.org / GitLab FOSS

Stan Hu requested to merge stanhu/gitlab-ce:current-user-to-top-filter into master Apr 09, 2015

What does this MR do?

This MR puts the current user at the top of the user list in the author/assignee filters.

Are there points in the code the reviewer needs to double check?

Is it best to do this in the JavaScript as it is in this MR, or are we better off adding an argument to the /users API to put the current user on top?

Also, I elected to put the current user after "Any" and "Unassigned"--it looked better. Do you agree?

Why was this MR needed?

If you're trying to look for "Me" in the list, often you have to scroll down a long way to find yourself. Using the filter to find yourself is yet another step requiring the keyboard, and it's easy to forget that feature is available.