feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

Core: improve highlighting of fuzzy action matches

Browse Source 
Split the highlight if it covers the border.

Change-Id: I9177abf18e988b8c2f79ab355a1f17ad3bda4286
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
This commit is contained in:
David Schulz
2022-09-26 10:55:09 +02:00
parent e7fb5276af
commit d143f0c2e1
1 changed files with 17 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/plugins/coreplugin/actionsfilter.cpp
View File

@@ -110,8 +110,8 @@ QList<LocatorFilterEntry> ActionsFilter::matchesFor(QFutureInterface<LocatorFilt
return FilterResult{MatchLevel::Normal, withHighlight(filterEntry)};
}
FuzzyMatcher::HighlightingPositions positions = FuzzyMatcher::highlightingPositions(
allTextMatch);
const FuzzyMatcher::HighlightingPositions positions
= FuzzyMatcher::highlightingPositions(allTextMatch);
const int positionsCount = positions.starts.count();
QTC_ASSERT(positionsCount == positions.lengths.count(), return {});
const int border = first == Highlight::DisplayName ? filterEntry.displayName.length()
@@ -126,8 +126,21 @@ QList<LocatorFilterEntry> ActionsFilter::matchesFor(QFutureInterface<LocatorFilt
type = first == Highlight::DisplayName ? Highlight::ExtraInfo
: Highlight::DisplayName;
} else if (start + length > border) {
// skip this highlight since it starts before and ends after the border
// between the concatenated strings
const int firstStart = start;
const int firstLength = border - start;
const int secondStart = 0;
const int secondLength = length - firstLength - 1;
if (first == Highlight::DisplayName) {
highlight.startsDisplay.append(firstStart);
highlight.lengthsDisplay.append(firstLength);
highlight.startsExtraInfo.append(secondStart);
highlight.lengthsExtraInfo.append(secondLength);
} else {
highlight.startsExtraInfo.append(firstStart);
highlight.lengthsExtraInfo.append(firstLength);
highlight.startsDisplay.append(secondStart);
highlight.lengthsDisplay.append(secondLength);
}
continue;
}
if (type == Highlight::DisplayName) {