Skip to content

add currentCursor fgHighLight #48

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.

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

Closed
wants to merge 1 commit into from

Conversation

exilewithdj
Copy link

原版本的数独当前光标仅仅用^来显示,修改后的版本用光标所在位置的蓝色高亮背景来突出光标位置。这样让游戏玩起来更加舒适,在快速移动光标时,不用费劲去寻找当前光标位置。
e45a1f0c-f0ed-4ddb-99b3-26f6375115fe
ebc3d5af-0fdf-4531-a185-39f7b7d37254

@@ -40,7 +40,7 @@ bool CBlock::isFull() const
}
return true;
}

/*
Copy link
Owner

Choose a reason for hiding this comment

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

旧实现直接删除吧

Color::Code bgLight = isCursor ? Color::BG_BLUE : Color::BG_DEFAULT;
Color::Code fgLight = isCursor ? Color::BG_BLUE : Color::BG_DEFAULT;

if ((i + 1) % 3 == 0) {
Copy link
Owner

Choose a reason for hiding this comment

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

这段逻辑的if嵌套太多,有点难理解了。可以先合并后再思考如何优化和重构。

.cache
.vs
Copy link
Owner

Choose a reason for hiding this comment

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

.vs建议移动到#IDE下

@mayerui
Copy link
Owner

mayerui commented May 20, 2025

已实现该特性,我会关闭这个pr。

@mayerui mayerui closed this May 20, 2025
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