Skip to content

fix(gamepad): 修正校準視覺與短震體感#17

Merged
perditavojo merged 2 commits into
mainfrom
dev
Apr 19, 2026
Merged

fix(gamepad): 修正校準視覺與短震體感#17
perditavojo merged 2 commits into
mainfrom
dev

Conversation

@perditavojo
Copy link
Copy Markdown
Contributor

  • 修正高 DPI 下 LS/RS 標籤在 500% 縮放時被圖形遮蔽的問題
  • 調整多組短震設定檔的強度與時長,提升弱震模式下的可感知性
  • 將需要主馬達啟動時間的短脈衝補到更穩定的範圍

- 修正高 DPI 下 LS/RS 標籤在 500% 縮放時被圖形遮蔽的問題
- 調整多組短震設定檔的強度與時長,提升弱震模式下的可感知性
- 將需要主馬達啟動時間的短脈衝補到更穩定的範圍
Copilot AI review requested due to automatic review settings April 19, 2026 07:31
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

修正控制器校準視覺化在高 DPI 縮放下的標籤遮蔽問題,並微調多組短震動設定檔的強度/時長以提升弱震模式的可感知性與穩定度。

Changes:

  • 調整多個短脈衝/收尾類震動設定檔的 Strength 與 Duration,使短震更容易跨過馬達啟動閾值且節奏更穩定。
  • 校準視覺化繪製流程加入/擴大 DPI scale 的套用範圍(margin、labelHeight、plotGap、LS/RS 標籤區域),避免在高縮放時被圖形干擾。

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/InputBox/Core/Feedback/VibrationPatterns.cs 更新多組 VibrationProfile 參數,提升短震可感知性與一致性。
src/InputBox/Core/Controls/GamepadCalibrationDialog.cs 將校準視覺化的多個排版/標籤尺寸改為 DPI-aware,改善高縮放遮蔽。

Comment thread src/InputBox/Core/Controls/GamepadCalibrationDialog.cs Outdated
@perditavojo
Copy link
Copy Markdown
Contributor Author

@copilot apply changes based on the comments in this thread

Agent-Logs-Url: https://github.com/rubujo/InputBox/sessions/2ce2af2a-8aa6-4f30-a2fa-5c2374039315

Co-authored-by: perditavojo <117562794+perditavojo@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 19, 2026

@copilot apply changes based on the comments in this thread

Done in 85c833c. Extracted plotVerticalPadding = 6f * s and used it in the plotSize calculation so the vertical padding now scales consistently with DPI alongside labelHeight and plotGap.

@perditavojo perditavojo merged commit 58e8b76 into main Apr 19, 2026
8 checks passed
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.

3 participants