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

Search in the Part Editor of parts with none category #234

Open
yalov opened this issue Jun 20, 2024 · 8 comments
Open

Search in the Part Editor of parts with none category #234

yalov opened this issue Jun 20, 2024 · 8 comments

Comments

@yalov
Copy link

yalov commented Jun 20, 2024

The known feature is that if the category is none, than a part is hidden from the parts search.

As I understand, an asteroid part or a kerbal part have a category equal to none and therefore cannot be searchable — that is good.

A problem is a conflict with "Community Category Kit" mechanic, what is getting worse with the Community Parts Titles Extras: CCK - No Duplicates — in this scenario, a part could be in the cck category, but be hidden from the stock category (has category set to none), and therefore not searchable.

One of the solution would be to search all parts, but hardcode exclude kerbals, asteroid and comet.
I am not familiar with other "special" parts (stock or mods) that need to be exluded.

@nagleaidan
Copy link

That sounds like a problem for CCK to fix, not this mod

@yalov
Copy link
Author

yalov commented Jun 20, 2024

The default behavior of disabling search for parts with no category is questionable and generally unexpected.

I believe parts should remain searchable even if they are hidden from all categories, whether intentionally or unintentionally, in the simple (one-column) view in the editor.

@Rodg88
Copy link

Rodg88 commented Jun 20, 2024

Having category = none is actually used to specifically hide parts in some cases, particularly when soft-deprecating parts to prevent their use without outright deleting them.

@nagleaidan
Copy link

nagleaidan commented Jun 20, 2024

The default behavior of disabling search for parts with no category is questionable and generally unexpected.

It's not an unexpected behavior if you're playing stock or with most mods (CCK being a notable exception). I don't want to search and see deprecated parts or asteroids or Jeb's Workshop (a JNSQ Easter egg).

I believe parts should remain searchable even if they are hidden from all categories, whether intentionally or unintentionally, in the simple (one-column) view in the editor.

This is a feature, you may disagree with it but it's still a feature. Community Fixes is designed for fixing bugs, not altering features. If you want to change this feature, make a separate mod.

@yalov
Copy link
Author

yalov commented Jun 20, 2024

I agree that CCK itself would be a better place for this setting — but that will never happen.
And CCK is very popular, based on CKAN download count.

Also I would add the full search in the "Community Parts Titles Extras: CCK - No Duplicates" if I would know how to work with Harmony.

Maybe It could be a KSPCommunityFixes setting, disabled by default, what could enable search for everything, even for asteroid, kerbalEVA, and soft-deprecating parts?

I still prefer to have several deprecated parts in the search results rather than be unable to find a properly loaded part due to some CCK-related patch incompatibility.

@JonnyOThan
Copy link
Contributor

I still prefer to have several deprecated parts in the search results rather than be unable to find a properly loaded part due to some CCK-related patch incompatibility.

Isn’t this solvable with a simple MM patch that changes the category of all parts that have it set to none?

@yalov
Copy link
Author

yalov commented Jul 6, 2024

Isn’t this solvable with a simple MM patch that changes the category of all parts that have it set to none

No, setting @category = anything_else with KSPCF makes a part load into the default category, which is fuel tanks. Without KSPCF, it doesn't load at all, so @category = anything_else does not work either.

@yalov yalov closed this as completed Jul 6, 2024
@yalov yalov reopened this Jul 6, 2024
@yalov
Copy link
Author

yalov commented Jul 6, 2024

Is it within the scope to create a new "hidden" enum that is hidden and does not appear as a category in the GUI?

if it will be also default (#159), then failed to load without KSPCF parts will be able to load, but will be not appearing in the Fiel Tanks category.

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

No branches or pull requests

4 participants