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 changing the order of facts in Overview #216

Open
jtojnar opened this issue Sep 15, 2017 · 2 comments
Open

Allow changing the order of facts in Overview #216

jtojnar opened this issue Sep 15, 2017 · 2 comments

Comments

@jtojnar
Copy link
Member

jtojnar commented Sep 15, 2017

Currently, the overview uses ORDER BY DATE(start) DESC, TIME(start) ASC. For many users, simple ORDER BY start DESC or ORDER BY start ASC will be more natural. We should allow them to customize it.

@floi
Copy link

floi commented Aug 18, 2019

It looks like facts per day aren't actually ordered by start, but by their database id/order they were saved in. There is a comment in the code suggesting that ordering by fact start is the intended behavior, though.

I'm interested in trying my hand at this, but making the order configurable entails a UI decision I don't feel qualified to make – should this be set via the preferences or by, say, a button or combobox in the header bar?

@floi
Copy link

floi commented Aug 30, 2019

After giving this some more thought, I think that users might not want to change the order all the time so that adding this to the preferences makes more sense.

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