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

Alphabetize friends/friends-of, group notifications, group owned/member lists #6284

Merged
merged 1 commit into from Dec 23, 2013
Merged

Alphabetize friends/friends-of, group notifications, group owned/member lists #6284

merged 1 commit into from Dec 23, 2013

Conversation

beck24
Copy link
Member

@beck24 beck24 commented Dec 23, 2013

Some lists ordered by time_created doesn't make any sense for the user. Sort these alphabetically instead.

@juho-jaakkola
Copy link
Member

I'm fine with these changes. I often get requests to make the lists ordered alphabetically.

@beck24
Copy link
Member Author

beck24 commented Dec 23, 2013

Ditto, and it's almost always these ones so lets make this default for 1.9

beck24 added a commit that referenced this pull request Dec 23, 2013
Alphabetize friends/friends-of, group notifications, group owned/member lists
@beck24 beck24 merged commit 3e5a7d3 into Elgg:master Dec 23, 2013
@ewinslow
Copy link
Contributor

Woo, thank you!
On Dec 22, 2013 10:05 PM, "Matt Beckett" notifications@github.com wrote:

Some lists ordered by time_created doesn't make any sense for the user.

Sort these alphabetically instead.

You can merge this Pull Request by running

git pull https://github.com/beck24/Elgg alphabetize-core-lists

Or view, comment on, or merge it at:

#6284
Commit Summary

  • Alphabetize friends/friends-of, group notifications, group
    owned/member lists

File Changes

Patch Links:

@beck24 beck24 deleted the alphabetize-core-lists branch February 1, 2014 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants