Skip to content

Conversation

Copy link

Copilot AI commented Oct 28, 2025

  • Understand the issue: Background opacity slider in settings doesn't affect window background
  • Identify root cause: SettingsService was Transient, causing different instances
  • Change SettingsService to Singleton lifetime
  • Modify MainViewModel to use SettingsService for opacity value
  • Make MainViewModel listen to navigation changes and update background brush
  • Test the changes to ensure the slider works correctly
  • Run code review and security checks
Original prompt

This section details on the original issue you should resolve

<issue_title>[Bug] 背景透明度滑块不生效问题</issue_title>
<issue_description>## 问题描述

1301版本和1300版本中设置里面的背景透明度设置疑似失效


复现步骤

详细说明复现问题的步骤:

  1. 打开应用
  2. 进入“设置”页面
  3. 修改背景透明度滑块
  4. 背景透明度不改变

期望的行为

背景透明的通过滑块被修改


实际的行为

暂无


运行环境

请提供你的系统和软件版本信息:

  • 操作系统:Window 11
  • 软件版本:v1.3.0.1 zhcn

日志文件

-

附加信息

</issue_description>

Comments on the Issue (you are @copilot in this section)

@iRyougi 转自:您好,我之前一直用的n网原博主的管理器,但是他的1301版本和1300版本中设置里面的背景透明度设置疑似失效了,原博主也一直没修,所以我换回了1210版本,因为这个版本可以正常生效,然后今天看到up自己修改的版本,用了一下发现背景透明度滑块也是不生效的,能麻烦up检查一下是否是管理器的设置出现了问题,还有就是视频里展示的mod更新按钮我这边也没看到,请问是我自己这边没设置好还是其他问题

ref(APA): iRyougi.神椿仮想世界研究開發部.https://www.iryougi.com. Retrieved 2025/10/29.</comment_new>

  • Fixes iRyougi/Helldivers2ModManager#2

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@iRyougi iRyougi marked this pull request as ready for review October 28, 2025 20:21
@iRyougi iRyougi merged commit 9014b1f into master Oct 28, 2025
1 check failed
@iRyougi iRyougi deleted the copilot/fix-background-opacity-slider branch October 28, 2025 20:25
Copilot AI requested a review from iRyougi October 28, 2025 20:25
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.

2 participants