Update navigation mesh baking doc #8591
Conversation
smix8
force-pushed
the
navigationregion_baking
branch
from
535bfe0 to
14cda27
Compare
skyace65
added
enhancement
area:manual
smix8
force-pushed
the
navigationregion_baking
branch
from
14cda27 to
674f471
Compare
timothyqiu
left a comment
timothyqiu
left a comment
There was a problem hiding this comment.
Some markers are used incorrectly.
smix8
force-pushed
the
navigationregion_baking
branch
2 times, most recently
from
5aa4ac9 to
cf46d62
Compare
AThousandShips
left a comment
AThousandShips
left a comment
There was a problem hiding this comment.
Think some of the sentences need to be broken up with commas or periods, to lessen the mental workload of reading long sentences, especially before connecting words such as "and" and "or"
smix8
force-pushed
the
navigationregion_baking
branch
from
cf46d62 to
8f0f083
Compare
skyace65
left a comment
skyace65
left a comment
There was a problem hiding this comment.
I don't think we have a specific line length limit. But the general rule of thumb is if it is so long it shows up on two lines in github it should be broken up.
Also for the suggested changes on "-" lines make sure the first letter of the second line lines up with the first letter on the first line. I think my suggested changes do but it's hard to tell.
smix8
force-pushed
the
navigationregion_baking
branch
from
8f0f083 to
9275322
Compare
|
@AThousandShips any other changes that should be made? |
timothyqiu
left a comment
timothyqiu
left a comment
There was a problem hiding this comment.
Did a proofreading. I think it's worth adding some punctuation marks.
Also found a typo. (agent_max_climp -> agent_max_climb)
AThousandShips
left a comment
AThousandShips
left a comment
There was a problem hiding this comment.
LGTM, but agreeing with the punctuation suggestions
smix8
force-pushed
the
navigationregion_baking
branch
from
9275322 to
7a168a5
Compare
Updates navigation mesh baking doc.
smix8
force-pushed
the
navigationregion_baking
branch
from
7a168a5 to
c8f7e92
Compare
|
Thanks! Great navigation docs as always. |
|
Cherry-picked to 4.2. |
5 participants
Updates navigation mesh baking doc.
Adds doc for the new 2D navigation mesh baking in Godot 4.2.
Adds examples for NavigationRegion and NavigationServer baking.
Removes the outdated descriptions for the old navigation mesh baking or deprecated functions.
There is a lot of stuff still missing but that will take another few weeks if not months and I don't want the doc to stay completely outdate with wrong information and examples for that long.