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.

Sign up for GitHub

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

Jump to bottom

Sorting order in visitor looks shuffled #9579

Open
hpvd opened this issue Jan 20, 2016 · 2 comments
Open

Sorting order in visitor looks shuffled #9579

hpvd opened this issue Jan 20, 2016 · 2 comments
Labels
c: Usability For issues that let users achieve a defined goal more effectively or efficiently.
Milestone
Backlog (Help wan...

Comments

@hpvd
Copy link

hpvd commented Jan 20, 2016

When looking in visitor looks the order of visits look shuffled.
Would expect when a time is shown (which is probably the time of the begin of the visit?)
this time is also used for chronological sorting. Other wise it looks broken.
please see attachment

2016-01-20_08h48_53

(this was also reported as side issue within #9504)
If one want to keep the oder, one should show the time used for sorting....
@hpvd hpvd changed the title Sorting order in visitor log shuffeld Sorting order in visitor looks shuffled Jan 20, 2016
@tsteur
Copy link
Member

tsteur commented Jan 20, 2016

I agree, had this thought a couple of times.

From a quick look they are ordered by "visit_last_action_time" when they are fetched from DB, the date shown seems to be from the first action indeed which I think is on purpose but I forgot why.

If they were ordered by first_action_time, and a user is active over a while, the user wouldn't be visible in the visitor log anymore. This means we cannot really sort by first_action_time but instead would need to show last_action_time in the UI. There was probably a reason why this one is not shown.

Anyhow, it must be made clearer somehow. I don't think we have to change what is shown but somehow explain it or make it more clear. Maybe it's even possible to show both first and last action date...

@hpvd
Copy link
Author

hpvd commented Jan 20, 2016

showing both would help!

maybe if visitor is still active
06:46:23 - now

otherwise
05:32:05 - 05:56:12

to hove enough room one could save some space (if needed) by

  • shortening day (Wednesday -> Wed.) and
  • show date as numbers (January 20, 2016 -> 20/01/2016)

all together:
Wednesday, January 20, 2016 - 05:32:05
->
Wed. 20/01/2016 05:32:05 - 05:56:12

@mattab mattab added the c: Usability For issues that let users achieve a defined goal more effectively or efficiently. label Mar 31, 2016
@mattab mattab added this to the Mid term milestone Mar 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: Usability For issues that let users achieve a defined goal more effectively or efficiently.
Projects
None yet
Milestone
Backlog (Help wanted)
Development

No branches or pull requests
3 participants
@tsteur @mattab @hpvd