-
-
Notifications
You must be signed in to change notification settings - Fork 21.6k
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
Doc items breaking into groups by first sentence rather than alphabetically #35347
Comments
Ah, maybe it's just a matter of taste... They are grouped by first sub-sentence rather than alphabetically, which may be ok. |
Hi, I would like to take on this issue, but not exactly sure on how assigning these work. I would be happy in helping to sort out the doc items alphabetically. |
@mdante97 I think the current way of sorting is fine. It's already alphabetical, it just happens to group results by their first sub-sentence like This issue can probably be closed. |
I think the grouping by prefix similarity could simply be removed now. I assume it was done back in the days when we didn't have properties and had dozens of |
Removing additional newline command in editor_help.cpp at line 655 (inside if condition) should do it. |
@Anukriti12 Feel free to open a pull request right away; you can rebase it in the future if it gets conflicts due to refactorings. I believe editor-related refactorings have mostly been completed by now (especially for parts that could affect the editor help). |
@Anukriti12 without newline, it's hard to read and find a method when they are a lot. |
something else, if we break by alphabetically the section Methods, we need to do the same for Properties which above Methods for the same reason |
Godot version:
3.1 - 3.2rc1
Issue description:
![image](https://user-images.githubusercontent.com/3036176/72701394-34fb3900-3b60-11ea-9a66-5b0b4235f036.png)
Looks like incorrect spacing where it shouldn't and not breaking where it should:
The text was updated successfully, but these errors were encountered: