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

Add option to show started/starting processes by default #207

Closed
stijnveenman opened this issue Jul 30, 2024 · 4 comments
Closed

Add option to show started/starting processes by default #207

stijnveenman opened this issue Jul 30, 2024 · 4 comments
Labels
done Done, awaiting release enhancement New feature or request

Comments

@stijnveenman
Copy link
Contributor

Feature Request

The feature of being able to see all processes, even those disabled is great. However, this leads to me basically always pressing Ctrl-D on startup. And only sometimes using it to start another process. It would be great to have a way to only show enabled by default

Use Case:

Proposed Change:

For me personally, i think defaulting to only showing enabled by default makes sense. However, this does change behaviour. So either adding an env var and startup argument (env var is my preferred method, so i don't have to always type the argument) would be sufficient as well.

Who Benefits From The Change(s)?

Anyone with a large number of processes that often only starts a small subset of processes.

Alternative Approaches

@F1bonacc1
Copy link
Owner

Hi @stijnveenman,

That's a good proposal.
Today PC already has the --hide-disabled flag. Unfortunately, it was added before the ability to hide (as opposed to remove) the disabled processes.
What I am thinking is to convert it to a "hide" instead of "remove" behavior. I believe that from a user perspective, it will look the same, and add the ability to "unhide" them with Ctrl-D.

What do you think?

@F1bonacc1 F1bonacc1 added enhancement New feature or request need more info labels Aug 3, 2024
@stijnveenman
Copy link
Contributor Author

I think that makes sense, also from a UX perspective it makes more sense with the introduction of the new feature.
Does this have an environment variable linked to it?

@F1bonacc1
Copy link
Owner

Does this have an environment variable linked to it?

No, but I will add it (actually already did)

@F1bonacc1 F1bonacc1 added done Done, awaiting release and removed need more info labels Aug 4, 2024
@F1bonacc1
Copy link
Owner

Added in v1.18.0

Cheers ✌🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done Done, awaiting release enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants