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

UI: don't use a separate ListView for active conversations #14

Open
mrDoctorWho opened this issue Oct 17, 2017 · 3 comments
Open

UI: don't use a separate ListView for active conversations #14

mrDoctorWho opened this issue Oct 17, 2017 · 3 comments

Comments

@mrDoctorWho
Copy link

Here's my UI suggestion.

Put current conversations and roster in a signle list view, separated by a horizontal line.

@horazont
Copy link
Contributor

I thought about merging the two lists a while back, but I’m not convinced that’s a good UI:

  1. I’d like to have open conversations easily accessible, without filtering or scrolling through the roster.
  2. I definitely not want to sort by most-recent-activity (most certainly not by default), because that allows third-parties to modify the sort order, leading to annoying (albeit probably harmless) clickjacking (think you’re clicking on a roster item to start a conversation and then the list re-sorts due to a new message from somebody).

This proposal here is a bit more appealing because it would simply add a group of conversations to the top of the contact list, without re-ordering the existing contact list. I don’t really like that because it mixes two very different things. The interaction with filters would have to be defined, as well as with possibly different sort orders.

I can be convinced. Let the flame begin.

@mrDoctorWho
Copy link
Author

I definitely not want to sort by most-recent-activity (most certainly not by default), because that allows third-parties to modify the sort order, leading to annoying (albeit probably harmless) clickjacking (think you’re clicking on a roster item to start a conversation and then the list re-sorts due to a new message from somebody).

Sure. I should have mentioned that, I think you just need to create new items (just in GUI) in roster, so it would not require you to mess with ordering.

@horazont
Copy link
Contributor

horazont commented Jul 3, 2018

Care to test the current master branch? I made some adjustments (video).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants