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.

Sign up for GitHub

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

Jump to bottom

[feature] 地图追踪/脚本添加重新加载按钮 #627

Closed
boxsnake opened this issue Sep 15, 2024 · 3 comments
Closed

[feature] 地图追踪/脚本添加重新加载按钮 #627

boxsnake opened this issue Sep 15, 2024 · 3 comments
Labels
UI/UX 已完成 已完成未发布

Comments

@boxsnake
Copy link

  • Your feature requesting / 你的新功能请求

由于在创建脚本/追踪JSON时，会通过直接修改文件进行编辑，但 BetterGI 无法监听到文件变更或目录变更。因此建议在这两个标签页中加入对应的重新加载按钮，以方便开发调试。
@huiyadanli
Copy link
Member

huiyadanli commented Sep 16, 2024 •
edited
Loading

切换标签页后,会刷新。

可以添加一个右键刷新的功能

huiyadanli added a commit that referenced this issue Sep 16, 2024
@huiyadanli
add refresh context menu #627
779fbaa 
添加刷新命令菜单项及刷新方法

在 `JsListPage.xaml` 和 `MapPathingPage.xaml` 中，添加了新的菜单项 `<MenuItem>`，用于绑定 `RefreshCommand` 命令，并设置了相应的 `CommandParameter` 和 `Header` 属性。

在 `JsListViewModel.cs` 中，添加了 `using BetterGenshinImpact.GameTask.AutoPathing.Model;` 语句，并修改了 `OnOpenScriptProjectFolder` 方法，将 `item` 为 `null` 时的处理逻辑改为使用 `scriptPath`。

在 `JsListViewModel.cs` 和 `MapPathingViewModel.cs` 中，添加了 `OnRefresh` 方法，该方法通过调用 `InitScriptListViewData` 来刷新脚本列表视图数据。
@huiyadanli huiyadanli added the 已完成 已完成未发布 label Sep 16, 2024
@boxsnake
Copy link
Author

由于界面上并没有提示说明切换标签页会刷新数据，所以用户一般不会认为切换标签页会刷新数据。

并且，刷新标签页刷新数据可能是导致调度器中当前选中配置组的选中状态消失的原因。

@huiyadanli
Copy link
Member

huiyadanli commented Oct 13, 2024 •
edited
Loading

https://github.com/babalae/better-genshin-impact/releases/tag/0.35

加了，但是改了UI以后又无了，后面会补回来

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
UI/UX 已完成 已完成未发布
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@boxsnake @huiyadanli