Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow filtering of user list by user status #296

Merged
merged 13 commits into from Dec 24, 2014
Merged

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Dec 23, 2014

https://www.agileplannerapp.com/boards/173808/cards/9014

Create a filter so that users can be viewed by the following statuses:

  • Suspended
  • Invited
  • Locked
  • Passphrase expired
  • Active

Changes:

  • Replace suspended column in user list with a status column
  • Ports filter styles from Transition app
  • Tidies up role filter UI, makes it easier to remove filters
  • Allows filtering by role, status, name/email at the same time
  • Adds a count for users displayed

Suggest reviewing with ?w=1 because of 36528f0

Filter by status

screen shot 2014-12-23 at 14 43 33

Filter by role and status

screen shot 2014-12-23 at 14 43 57

Statuses

screen shot 2014-12-23 at 14 45 35

fofr added 13 commits Dec 22, 2014
* See whether users are active, suspended, invited or if their
passphrase has expired
Give more space to filters before adding new filter
Abstract out the filter type to make management of filters easier and
simplify markup.
Bring across white filter styles from Transition. These should
eventually live in the admin template.

* Introduce a nag-compact style which has less padding
* Highlight active filters
* Introduce a remove filter button
* Reduce size of chevron in filter
* Add ‘pill’ styles for text only items in a nav-pill container
Quickly return to the original list of users
When titles are floated or within grids/containers, the margins
collapse differently, which pushed the title further down the page than
it ought to be.
vinayvinay added a commit that referenced this pull request Dec 24, 2014
Allow filtering of user list by user status
@vinayvinay vinayvinay merged commit 78a750a into master Dec 24, 2014
1 check passed
1 check passed
default "Build #421 succeeded on Jenkins"
Details
@vinayvinay vinayvinay deleted the user-account-counts branch Dec 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.