search

herbix / hoi4modutilities

A VSCode extension that contains utilities for Heart of Iron IV mods developers
MIT License
83 stars 10 forks source link

dynamic preview localisation #94

Closed IShiraiKurokoI closed 2 months ago

IShiraiKurokoI commented 2 months ago

features

The effect preview can be seen on my fork https://github.com/IShiraiKurokoI/hoi4modutilities/blob/master/README.md About the readDirFilesRecursivelyImpl https://ishiraikurokoi.top/2024-09-01-VSCode-Extension-Directory-Walk/ I'm not very good at writing typescript, so please forgive me if there are any non-standard parts.

Edited: Dynamic preview localization means that when you modify the properties of the corresponding node, the localized display of the preview corresponding to the right node will change accordingly at the same time. If you modify the global preview language setting or change the localisation file, you will need to close the preview panel and reopen it for it to take effect.(Maybe this could be inproved in future)

herbix commented 2 months ago

Awesome feature! I'll take a look when I have time.

herbix commented 2 months ago

@IShiraiKurokoI 感谢你对这个工具的改进！我对这个PR有几个意见:

  1. 你把很多 {[空格]内容[空格]} 里面的空格去掉了，我不知道是否是自动工具做的。我希望能保持代码风格统一，你能不能把这些都恢复成原来的样子？
  2. 有一些注释是用中文的，可以改成英文吗？
  3. ko和ru的本地化，因为我不懂，我不知道你是否用了机翻。如果是的话，我比较倾向说不翻译，而不是用机翻。
IShiraiKurokoI commented 2 months ago

@IShiraiKurokoI 感谢你对这个工具的改进！我对这个PR有几个意见:

  1. 你把很多 {[空格]内容[空格]} 里面的空格去掉了，我不知道是否是自动工具做的。我希望能保持代码风格统一，你能不能把这些都恢复成原来的样子？
  2. 有一些注释是用中文的，可以改成英文吗？
  3. ko和ru的本地化，因为我不懂，我不知道你是否用了机翻。如果是的话，我比较倾向说不翻译，而不是用机翻。

代码风格的问题，应该是Idea保存的时候自动格式化导致的。然后剩下的问题的话我晚上下班之后修改一下重新提PR。

herbix commented 2 months ago

@IShiraiKurokoI 你不用close PR，直接在你的branch上更新就会同步过来。

IShiraiKurokoI commented 2 months ago

@IShiraiKurokoI 你不用close PR，直接在你的branch上更新就会同步过来。

@herbix 我主要是觉得之前的pr的commit太乱了，于是重置了后整理了一下pr branch，所以这个原来的不能自动同步只能close开一个新的了。