-
Notifications
You must be signed in to change notification settings - Fork 13
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
1.4.0 - Bakery types & fixed hanging Bakery issue #32
Conversation
Types now check for bakery in project, prevents errors if bakery not in project Added bakery to assembly definition or else cannot be seen Renamed types to better fit format
This reverts commit 011d4fb.
Added a setting to disable the prompt that asks if you want to apply scene changes
Fixed some spacing
Fixed nullref when setting up EQS for the first time
Added Bakery Lightmapper light types
vrchat.jordo.easyquestswitch/Runtime/Resources/EQS_Localizations/es.json
Outdated
Show resolved
Hide resolved
@@ -21,6 +21,7 @@ | |||
"SettingsHierarchy": "层次结构选项", | |||
"SettingsHierarchyIconShow": "显示图标", | |||
"SettingsHierarchyIconOffset": "隐藏图标", | |||
"SettingsPromptForPlatformChange": "Show prompt to apply EQS changes if a newly opened scene still uses setting from the opposite platform (Can cause issues with Bakery).", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CallMeReznov If you are available, would you have a translation to suggest for this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have roughly read your code in "EQS Window. cs"
Can I understand it as,
"If I open a scene in the query platform settings under the Windows platform settings, the EOS switching prompt will pop up."?
Can you upload a picture with this prompt appearing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is to enable/disable a popup that shows when:
- You setup EQS in one scene
- You switch to another scene
- You change platform (eg from PC to Android)
- You switch back to the original scene where you setup EQS
Because you didn't change platform while you were in the original scene, the original scene is still using PC settings, so a prompt comes up to ask you if you'd like to apply the Android settings like so:
You've already translated the prompt here:
EasyQuestSwitch/vrchat.jordo.easyquestswitch/Runtime/Resources/EQS_Localizations/cn.json
Line 27 in fe1c02a
"PopupTargetChanged": "自上次打开此场景以来,生成目标已更改,是否为{0}应用EQS变更?\n这可以在设置菜单中手动完成。", |
This is to enable/disable that feature in the settings, located right here:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got it, actually there is no need for such a complex translation
I roughly organize it as
"Is the EQS pop-up application change prompt enabled for scene changes(Remember to manually apply the changes after closing, otherwise it may cause baking problems)"
"SettingsPromptForPlatformChange": "是否打开场景更改时的EQS弹出的应用变更提示 (关闭后需要手动应用变更,否则可能造成烘培错误).",
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MMmmm, the thing that causes issues with Bakery is the prompt itself. The prompt appearing makes Bakery hang indefinitely when you try to bake until you interact with it, so I'm not sure this translation would be accurate? This is specifically to enable/disable the prompt showing up and giving a slight warning that it may cause Bakery to hang.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the popup is useful for some, but for users of the tool Bakery https://assetstore.unity.com/packages/tools/level-design/bakery-gpu-lightmapper-122218 it can cause an issue and hang the process of baking, so this toggle allows them to turn off the popup, if you can re-word the issue to be about Bakery specifically I think it'd be perfect 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"SettingsPromptForPlatformChange": "是否打开场景更改时EQS弹出的应用变更提示 (提示功能与Bakery工具同时启用时,在烘培过程中弹出提示会造成烘培挂起,需手动点击弹窗后恢复).",
Following your suggestion, I have optimized the translation again
However, I personally believe that there should not be too much text introduction added in this position. On the one hand, I will optimize and abbreviate the text for the introduction of the GUI, and on the other hand, I want to provide a detailed introduction to the origin of the problem
These are two extremes
I suggest adding hyperlinks and creating a page within the plugin's GitHub to carefully explain the issue
For example:
"SettingsPromptForPlatformChange": "是否打开场景更改时EQS弹出的应用变更提示 (与Bakery存在兼容性问题 [详情请点击]#32)."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mmm, do you think it's necessary to explain the whole reason on why it's not working? I thought just having "Can cause issues with Bakery" would be enough, and if you have Bakery and get the issue you'll know what this is talking about instantly (as your bake stops until you acknowledge the popup).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just an improvement suggestion, and there are no issues with the current version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would "是否打开场景更改时EQS弹出的应用变更提示 (与Bakery存在兼容性问题)." be fine then?
Thank you for taking the time to translate this with me, I really appreciate it 😄
vrchat.jordo.easyquestswitch/Runtime/Resources/EQS_Localizations/jp.json
Outdated
Show resolved
Hide resolved
Added Spanish localization for SettingsPromptForPlatformChange
No description provided.