Skip to content
This repository was archived by the owner on Feb 3, 2026. It is now read-only.

feat: 焦点列表增加隐藏焦点方法#75

Merged
tinymins merged 2 commits intotinymins:masterfrom
LyKoiya:master
Aug 18, 2025
Merged

feat: 焦点列表增加隐藏焦点方法#75
tinymins merged 2 commits intotinymins:masterfrom
LyKoiya:master

Conversation

@LyKoiya
Copy link
Contributor

@LyKoiya LyKoiya commented Aug 15, 2025

焦点列表:增加隐藏焦点方法,当满足规则(血量百分比、目标关系)时,判断最大距离(nMaxDistance)是否为-1,若为-1则不显示焦点并跳出后续规则遍历。

@tinymins tinymins force-pushed the master branch 2 times, most recently from 5065126 to 12dc7fc Compare August 15, 2025 20:02
@tinymins tinymins requested a review from Copilot August 15, 2025 20:11
Copy link

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

This PR adds functionality to hide focus targets in the focus list when specific conditions are met. The change implements a rule-based focus hiding mechanism that checks if the maximum distance (nMaxDistance) is set to -1, and if so, hides the focus target and skips subsequent rule processing.

  • Added conditional logic to hide focus targets when nMaxDistance equals -1
  • Modified the focus display logic to respect the hide focus rule
Comments suppressed due to low confidence (1)

MY_Focus/src/MY_Focus.lua:885

  • The comment contains garbled characters that appear to be encoding issues. The Chinese text should be properly encoded to display correctly.
							--2025.8.16�������ؽ��㷽������������������ж��������Ƿ�Ϊ-1����Ϊ-1����ʾ���㲢�����������

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@LyKoiya LyKoiya changed the title fix: 焦点列表增加隐藏焦点方法 feat: 焦点列表增加隐藏焦点方法 Aug 17, 2025
@LyKoiya
Copy link
Contributor Author

LyKoiya commented Aug 17, 2025

已按提示修改标题为fix:

@tinymins tinymins merged commit 480ec35 into tinymins:master Aug 18, 2025
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments