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 class icons to script list, separate doc list #96956

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aXu-AP
Copy link
Contributor

@aXu-AP aXu-AP commented Sep 13, 2024

Implements godotengine/godot-proposals#3557 (as per further suggestions in the thread comment)
Alternate to #96865

kuva

It's not perfect, but can be used to get feel how this suggestion works.

@Maran23
Copy link
Contributor

Maran23 commented Sep 13, 2024

This is a nice idea! Lets see what other people think, but I like it! :)

@Mickeon
Copy link
Contributor

Mickeon commented Sep 14, 2024

The only "gripe" with this is that the documentation pages can no longer be shuffled around in the list. Gonna be honest, though I don't see why you'd mix-and-match around, given that the pages appearing at the bottom is much more appealing, and they are ordered to the bottom by default anyhow.

@Mickeon
Copy link
Contributor

Mickeon commented Sep 14, 2024

Also, by all means, see also this comment I don't forget on a related discussion.

@aXu-AP
Copy link
Contributor Author

aXu-AP commented Sep 14, 2024

Also, by all means, see also this comment I don't forget on a related discussion.

Heh, it seems to be the very same @seppoday who suggested this scripts/docs separation as well 😁
For collapsable structure, moving to Tree would give flexibility needed. In that case split wouldn't be needed to be hard coded either (altough problem of differentiating docs from scripts would return without splitting).

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

Successfully merging this pull request may close these issues.

4 participants