Skip to content

Commit d3743e5

Browse files
IshmaeelJan Wilmans
authored andcommitted
Remove color support from "Include" filter.
I would very much like to use the "include" filter but the fact that it has precedence over dynamic process colors has been a deal breaker. Now include filter does not apply text coloring either, very much like the exclude filter. Not a big functionality loss because it is still possible to recolor included lines by defining an additional highlight rule. Perhaps more work, but definitely more flexibility.
1 parent d660502 commit d3743e5

File tree

2 files changed

+1
-1
lines changed

2 files changed

+1
-1
lines changed

application/DebugViewpp/FilterPage.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ bool SupportsColor(FilterType::type filterType)
4949
switch (filterType)
5050
{
5151
case FilterType::Exclude:
52+
case FilterType::Include:
5253
return false;
5354
default:
5455
return true;

application/DebugViewpp/LogView.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2107,7 +2107,6 @@ bool FilterSupportsColor(FilterType::type value)
21072107
{
21082108
switch (value)
21092109
{
2110-
case FilterType::Include:
21112110
case FilterType::Highlight:
21122111
case FilterType::Track:
21132112
case FilterType::Stop:

0 commit comments

Comments
 (0)