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

Sort starred messages to the top of the list #127

Closed
wants to merge 1 commit into from

Conversation

danirabbit
Copy link
Member

Following up on https://bugs.launchpad.net/pantheon-mail/+bug/1590976

Not so sure about this anymore for a couple of reasons:

  1. There's a starred folder in the sidebar so these messages are already easy to get to
  2. The listbox lazy loads so it won't show all starred messages from ever, only recent ones

@danirabbit danirabbit added Needs Design Waiting for input from the UX team Rewrite labels Aug 2, 2017
@davidmhewitt
Copy link
Member

The starred folder only exists if your mailserver has one. I feel like that's a gmail thing.

@danirabbit
Copy link
Member Author

I guess we could enforce that there is always a starred folder if we don't find one?

@tintou
Copy link
Member

tintou commented Aug 2, 2017

Camel supports virtual folders so it's possible to create one if there is none

@danirabbit
Copy link
Member Author

Yeah I kinda am leaning towards enforcing the folder to be the better solution here. That way we're not cluttering up the inbox (Inbox 0 is the mission amirite?) and there's still an easy way to get to these messages

@danirabbit
Copy link
Member Author

Gonna close this since it sounds like there's a general consensus that this isn't a great workflow for everyone

@danirabbit danirabbit closed this Aug 2, 2017
@danirabbit danirabbit removed the Needs Design Waiting for input from the UX team label Aug 2, 2017
@danirabbit danirabbit deleted the starred-to-top branch August 2, 2017 20:43
@cassidyjames
Copy link
Contributor

I really like the starred-first approach (and use it in FastMail for my work email), but I agree that it might not be a generic enough thing to do by default. Perhaps down the road it could return as an option or something (having them sorted up top makes them unignorable, which is nice for Inbox Zero). 🤷‍♂️

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.

4 participants