Skip to content

Conversation

linbingquan
Copy link
Contributor

No description provided.

@lafflan
Copy link
Contributor

lafflan commented Nov 1, 2018

你好,我觉得有些单词还可以专业些,我列举我所知道的:
roughness 粗糙度
metalness 金属度
emissive 自发光
emissivemap 自发光贴图
displacemap 置换贴图
aomap 环境光遮蔽(罩)贴图
back 背面
shininess 高光大小
fragment 片元

@linbingquan
Copy link
Contributor Author

@sweerwen Thanks!

linbingquan@dbe72d6

var optionTxt = '???';

if ( key === 'WebGLRenderer' ) {
optionTxt = strings.getKey( 'sidebar/project/renderer/webglrenderer' )
Copy link
Owner

Choose a reason for hiding this comment

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

I wonder if we should be translating API names. Would people know that it's using THREE.WebGLRenderer when reading WebGL渲染器?

Copy link
Owner

Choose a reason for hiding this comment

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

Same thing for the 添加 section... Wouldn't it be better to keep the english names Group, Plane, Box, ... so people understand they're adding a THREE.Group, THREE.Plane, THREE.Box?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wonder if we should be translating API names. Would people know that it's using THREE.WebGLRenderer when reading WebGL渲染器?

I think most of people don't know THREE.WebGLRenderer when reading WebGL渲染器, maybe people think Render(渲染器) has relationship with WebGL. I don't commit this part of code until we found the better way.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same thing for the 添加 section... Wouldn't it be better to keep the english names Group, Plane, Box, ... so people understand they're adding a THREE.Group, THREE.Plane, THREE.Box?

This is other way like Group -> 组 Group, Plane -> 平面 Plane, Box -> 正方体 Box, but I don't think it is the best way.

How about we add some tips in options, like Blender software.

image

Copy link
Owner

Choose a reason for hiding this comment

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

That'd be nice indeed. We can do that in the future 😀

@mrdoob mrdoob added this to the r99 milestone Nov 1, 2018
@mrdoob mrdoob merged commit a554c8a into mrdoob:dev Nov 2, 2018
@mrdoob
Copy link
Owner

mrdoob commented Nov 2, 2018

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.

3 participants