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

JBIDE-21674 Enable hierarchical view by default in Project Explorer #487

Closed
wants to merge 1 commit into from

Conversation

scabanovich
Copy link
Contributor

No description provided.

@mickaelistria
Copy link
Contributor

A quick overview makes that this approach seems good: doesn't require any upstream change, and allows user to easily use the legacy flat project layout of they want to. So +1 on the design.
I won't have the opportunity to actually try that and get into details of the code before the 22nd. So if someone else validates it and there is a consensus that it is a good thing, +1 to merge it.

@maxandersen
Copy link
Member

without tests I'm -1 on this kind of change this late in the game.

Requiring our base/foundation plugins participating in startup and toggling these kind of settings is really not something that feels safe imo.

@robstryker
Copy link
Member

Very old issue and I agree with Max. Closing out.

@robstryker robstryker closed this Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants