mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-25 04:56:03 +03:00
[GH-ISSUE #394] The advanced search should show the most recent emails first (when too many results) #327
Labels
No labels
2fa
I18N
PGP
Security
Security
account
advanced_search
advanced_search
announcement
api_login
authentication
awaiting feedback
blocker
bug
bug
bug
calendar
config
contacts
core
core
devops
docker
docs
duplicate
dynamic_login
enhancement
epic
feature
feeds
framework
github
github
gmail_contacts
good first issue
help wanted
history
history
imap
imap_folders
inline_message
installation
keyboard_shortcuts
keyboard_shortcuts
ldap_contacts
mobile
need-ssh-access
new module set
nux
pop3
profiles
pull-request
question
refactor
release
research
saved_searches
smtp
strategic
tags
tests
themes
website
wordpress
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/cypht#327
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @marclaporte on GitHub (Apr 29, 2020).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/394
Originally assigned to: @jasonmunro on GitHub.
I accept that there are more results than what goes on page 1, but the order is not what I would expect.
@dumblob commented on GitHub (Apr 29, 2020):
I second this proposal as the order shall be the same for most email listings (for better UX).
@jasonmunro commented on GitHub (May 4, 2020):
Not sure I follow. Are the results matching the search criteria not in arrival order?
@marclaporte commented on GitHub (May 4, 2020):
Indeed.
@jasonmunro commented on GitHub (May 5, 2020):
my bad, I misunderstood the problem. The result limit slice is not starting at the right place because the results are unsorted when the slice occurs. Fix pushed to master
@jasonmunro commented on GitHub (May 5, 2020):
@marclaporte I will leave this open until you can confirm the fix is good for you. Thanks!
@marclaporte commented on GitHub (May 6, 2020):
Ok, thanks to Packagist.org and Tiki CI, this will automatically be in Tiki within a few days, so I will retest then.
Thanks!
@marclaporte commented on GitHub (May 12, 2020):
I confirm the fix. Thank you!