Page MenuHome
Differential D10618

UI: Prefer shorter search items in fuzzy search.
ClosedPublic
Actions

Authored by Jacques Lucke (JacquesLucke) on Mar 4 2021, 5:43 PM.

Details

Reviewers
Hans Goudey (HooglyBoogly)
Commits
rBf951aa063f7a: UI: prefer shorter search items in fuzzy search
Summary

Before:


After:


Diff Detail

Repository
rB Blender

Event Timeline

Jacques Lucke (JacquesLucke) requested review of this revision.Mar 4 2021, 5:43 PM
Jacques Lucke (JacquesLucke) created this revision.
Harbormaster completed remote builds in B13258: Diff 34765.Mar 4 2021, 5:43 PM
Jacques Lucke (JacquesLucke) edited the summary of this revision. (Show Details)Mar 4 2021, 5:44 PM
Jacques Lucke (JacquesLucke) edited the summary of this revision. (Show Details)Mar 4 2021, 5:47 PM
Jacques Lucke (JacquesLucke) added a reviewer: Hans Goudey (HooglyBoogly).
Hans Goudey (HooglyBoogly) accepted this revision.Mar 4 2021, 6:14 PM

This seems like a natural improvement, and in testing I'm not getting any unexpected results. Well really I'm getting more expected results.

After taking a minute to understand the code here (had to notice that it was a multi-value map for it to make sense), the code looks straightforward too.

This revision is now accepted and ready to land.Mar 4 2021, 6:14 PM
Closed by commit rBf951aa063f7a: UI: prefer shorter search items in fuzzy search (authored by Jacques Lucke (JacquesLucke)). · Explain WhyMar 4 2021, 6:41 PM
This revision was automatically updated to reflect the committed changes.
Jacques Lucke (JacquesLucke) added a commit: rBf951aa063f7a: UI: prefer shorter search items in fuzzy search.
Campbell Barton (campbellbarton) awarded a token.Mar 5 2021, 3:45 AM

Revision Contents
Changeset List

PathSize
source/
blender/
blenlib/
intern/
17 lines

Diff 34769

View Options

source/blender/blenlib/intern/string_search.cc

Loading...