Skip to content

Sorting of networks within a protocol based on name#4054

Merged
ara4n merged 1 commit intoelement-hq:developfrom
ollieh:issues/3771
Oct 14, 2017
Merged

Sorting of networks within a protocol based on name#4054
ara4n merged 1 commit intoelement-hq:developfrom
ollieh:issues/3771

Conversation

@ollieh
Copy link
Copy Markdown
Contributor

@ollieh ollieh commented May 27, 2017

#3771

Now it looks like this:
screenshot_2017-05-27_17-27-39

@turt2live
Copy link
Copy Markdown
Member

iirc "matrix.org" is intended to be the parent heading, then all the things with icons are subpoints of matrix.org, just badly shown as such.

@ollieh
Copy link
Copy Markdown
Contributor Author

ollieh commented May 27, 2017

Will there ever be multiple headings, or can I just stick it at the top?

@turt2live
Copy link
Copy Markdown
Member

It's possible for custom-homeserver folk:
image

(and for others who define their settings correctly... somehow).

@ollieh
Copy link
Copy Markdown
Contributor Author

ollieh commented May 27, 2017

Ooh, that makes things rather trickier.

@ollieh
Copy link
Copy Markdown
Contributor Author

ollieh commented May 27, 2017

Changed it to just sort between headings, although it feels a bit complicated. I guess it could be simplified if it did this while it was adding them, but I wanted to keep the sorting logic separated in case I changed the behaviour.

Signed-off-by: Oliver Hunt <oliver@hunt.bz>
@ollieh
Copy link
Copy Markdown
Contributor Author

ollieh commented May 27, 2017

Figured out how the list was being constructed and put the sorting logic in there, much neater!
screenshot_2017-05-27_19-12-25

@ollieh ollieh changed the title Sorting of networks based on icon and text Sorting of networks within a protocol based on name May 27, 2017
@t3chguy
Copy link
Copy Markdown
Member

t3chguy commented May 27, 2017

looks much better than the initial pr diff

@lukebarnard1
Copy link
Copy Markdown
Contributor

Removing my assignment for now as I don't actually know if we want Gitter all the way down there. @ara4n's input would be appreciated.

@uhoreg uhoreg mentioned this pull request Jun 30, 2017
@ara4n
Copy link
Copy Markdown
Member

ara4n commented Oct 14, 2017

sorry for missing this months ago - if you need my input please assign it to me... the alphabetised list looks good to me.

@ara4n ara4n merged commit 6f81960 into element-hq:develop Oct 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants