Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Fix various sorting problems with url suggestions #8994

Closed
bbondy opened this issue May 22, 2017 · 0 comments
Closed

Fix various sorting problems with url suggestions #8994

bbondy opened this issue May 22, 2017 · 0 comments

Comments

@bbondy
Copy link
Member

bbondy commented May 22, 2017

  • Handle bookmarks that appear twice in folders with different counts.
  • Handle comparing items with no counts.
  • Handle comparing items with no lastAccessedTime
  • Handle sorting 2 sites when one is a substring of the other, match the
    shorter one first.
  • Fixed an issue with Immutable data sometimes being used in the
    sorting functions, and adds an assert to make sure it's correctly
    used.
  • Support javascript: bookmarklets
  • Avoid re-sorting suggestions that are generated
  • Adds lots of tests
@bbondy bbondy self-assigned this May 22, 2017
@bbondy bbondy added this to the 0.15.300 milestone May 22, 2017
@bbondy bbondy closed this as completed in da41b7e May 22, 2017
bbondy added a commit that referenced this issue May 22, 2017
Fix #8994

Auditors: @bsclifton

- Handle bookmarks that appear twice in folders with different counts.
- Handle comparing items with no counts.
- Handle comparing items with no lastAccessedTime
- Handle sorting 2 sites when one is a substring of the other, match the
  shorter one first.
- Fixed an issue with Immutable data sometimes being used in the
    sorting functions, and adds an assert to make sure it's correctly
    used.
- Support `javascript:` bookmarklets
- Avoid re-sorting suggestions that are generated
- Adds lots of tests
bbondy added a commit that referenced this issue May 25, 2017
Fix #8994

Historically there was some code that set 0 to null since we used to use 1 based ordinals instead of 0 based indexes, and it causes a bug where the first item can't be selected.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants