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

Docs: Object3D&SpotLight&TorusGeometry #22906

Merged
merged 2 commits into from
Nov 29, 2021
Merged

Docs: Object3D&SpotLight&TorusGeometry #22906

merged 2 commits into from
Nov 29, 2021

Conversation

jynxio
Copy link
Contributor

@jynxio jynxio commented Nov 28, 2021

Fixed #22827.

Description

1.Object3D
Change: The value type of children property is changed from Object3D to Array.
Reason: The value type of the children property is Array, it is described as Object3D in the document.
Target: zh, en, ko

2.SpotLight
Change: In the description of the target property, the term is changed from 平行光 to 聚光灯 .
Reason: 平行光 stands for DirectionalLight, 聚光灯 stands for SpotLight. Obviously, 聚光灯 shoudle be used here.
Target: zh

3.TorusGeometry
Change: In the parameters of 构造器, the Chinese annotations of radius and radialSegments are changed.
Reason: #22827 (you can see the reason and change in the second comment)
Target: zh

About Object3D:
The value type of the children property is Array, but the documents of zh, en, ko all define it as Object3D.

About SpotLight:
In the description of the target attribute, the SpotLight is translated into a DirectionalLight, this should be an unintentional mistake. (the error only occurs in Chinese documents).
Correct the definition of `radius` and 'radiusSegments' in '构造器'.
@jynxio jynxio changed the title Docs: Object3D&SpotLight Docs: Object3D&SpotLight&TorusGeometry Nov 28, 2021
@@ -35,7 +35,7 @@ <h3>[property:AnimationClip animations]</h3>
<h3>[property:Boolean castShadow]</h3>
<p>Whether the object gets rendered into shadow map. Default is *false*.</p>

<h3>[property:Object3D children]</h3>
<h3>[property:Array children]</h3>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At some point, we should document such properties or return types as: [property:Array<Object3D> children].

Unfortunately, the docs do no support such a markup yet. Maybe we can add something like this in a future PR?

@Mugen87 Mugen87 added this to the r136 milestone Nov 29, 2021
@Mugen87 Mugen87 merged commit 8cd4b99 into mrdoob:dev Nov 29, 2021
@mrdoob
Copy link
Owner

mrdoob commented Nov 29, 2021

Thanks!

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

Successfully merging this pull request may close these issues.

TorusGeometry's Chinese translation error
3 participants