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

menu 名と dialog 名を整理 #2184

Merged
merged 3 commits into from
Nov 15, 2023
Merged

Conversation

ousttrue
Copy link
Contributor

  • menu 名を関連クラスの public const string MENU_NAME にした(特に validate=true で2回同名で呼ぶところ)
  • menu 表示名と dialog のタイトルを同じ文字列にした

- menu 名を関連クラスの `public const string MENU_NAME` にした(特に validate=true で2回同名で呼ぶところ)
- menu 表示名と dialog のタイトルを同じ文字列にした
@ousttrue
Copy link
Contributor Author

gltfmenu
vrm0menu
vrm1menu

@@ -4,18 +4,12 @@

namespace UniGLTF
{
public static class TopMenuImplementation
Copy link
Contributor Author

Choose a reason for hiding this comment

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

関数が一個だけになったので、それに名前を合わせ

@@ -7,8 +7,7 @@ namespace UniGLTF.MeshUtility
{
public class MeshUtilityDialog : EditorWindow
{
const string TITLE = "Mesh Utility Window";

public const string MENU_NAME = "glTF MeshUtility";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

(glTF|vrm-0.x|vrm-1.0) (name) という感じの命名で統一してみた

TODO: language server 設定確認...
Copy link
Contributor

@Santarh Santarh left a comment

Choose a reason for hiding this comment

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

統一は LGTM なんですが、そのメニュー名に関しては修正したいです。

ユーザが何を目的にメニューを選択するのか?というと、たとえば「モデルを出力したいから」になります。
したがって、メニュー名は Export と動詞になっているべきです。
保存メニューが Save As ... なのと同じです。
ユーザは VRM Exporter というものは知りませんし、それを呼び出したくてこのメニューを選択するわけではない、と考えます。

また対話型 UI を呼び出すメニューなので末尾には ... をつけたいところです。
したがってメニュー名は以下のようにしたいです。

Export glTF...
Import glTF... (*.gltf|*.glb|*.zip)

Export VRM 0.x...
Import VRM 0.x...

Export VRM 1.0...
Import VRM 1.0...

また VRM のバージョン付き表記が VRM 1.0 なのか VRM-1.0 なのかは、VRM 公式サイトを見てもはっきりしませんが、この際、 glTF 2.0 に倣って VRM 1.0 VRM 0.x としたいです。

- `-1.0` => ` 1.0`
- `Import` and `Export`
- `Export < Import < MeshUtility` order
@ousttrue
Copy link
Contributor Author

gltfmenu_mod
vrm1menu_mod
vrm0menu_mod

@ousttrue
Copy link
Contributor Author

VRM-0.x Freeze T-Pose は MeshUtiility 内に取り込まれて、メニューからは消えるかもしれない

Copy link
Contributor

@Santarh Santarh left a comment

Choose a reason for hiding this comment

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

LGTM。

ただできればなんですが、ExportImport と MeshUtility の間に区切り線は欲しいかも。
殆どのユーザにとっては Export/Import 以外のメニューは重要度は低いはず。
次整理するならその時で。

VRM-0.x Freeze T-Pose は MeshUtiility 内に取り込まれて

@ousttrue
Copy link
Contributor Author

MeshUtility の前に区切り線は次で

@ousttrue ousttrue merged commit a700798 into vrm-c:master Nov 15, 2023
1 check passed
@ousttrue ousttrue deleted the fix_menu_name branch January 24, 2024 06:18
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.

None yet

2 participants