Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Smart Filtering: local & remote filters + colorized output #66

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

alacasta
Copy link
Contributor

@alacasta alacasta commented Dec 26, 2020

What?

This contribution fixex #57 and #59 and gathers old #60 and #58

The contribution includes

  • A new checkbox to enable the filtering remotely or in the client side
  • The modification of the non-matching logging to light-gray
  • The creation of a colorized filter

Colorized output

To create a filter

  • Add a new text in the filterbox
  • Click on Set Color and a color-picker will be open with a random color (see screenshot)
  • Select your preferred color
  • Click on Filter

image

Result

In case of having the remote filter enabled, the filter is made in remote

image

If in turn, thhe remote filter is disabled, the filter is made locally and the non-matching lines are displayed in gray

image

@alacasta alacasta changed the title Feature/local remote filters Feature: Smart Filtering: local & remote filters + colorized output Dec 26, 2020
@alacasta alacasta marked this pull request as ready for review December 26, 2020 11:12
@alacasta
Copy link
Contributor Author

alacasta commented Dec 26, 2020

Hello @pentix
Now seems that docker image comes with Qt 5.12

I propose you again to reconsider merging this feature but please, if you prefer to not add features from >= Qt 5.9.5, i can try to remove the dependency comming from QRandomGenerator

image

Let me know :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant