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.

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

feat: press enter to open the 1st result #163

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

BLumia
Copy link
Member

@BLumia BLumia commented Apr 10, 2024

行为调整,当正在搜索时,回车将直接启动结果中被选中的结果(即第一个结果),而不再是将焦点移到搜索区域。注意,当前实现仍与4/9讨论的预期行为不同,预期是希望搜索过程中也始终显示选中项,即搜索结果区域无焦点时,选中项也有一个近似于hover效果的指示效果的。

未在搜索时的行为不变(即,如果焦点在搜索框但没输入任何关键词就按下回车,此时会将焦点切换到应用区域)。

@BLumia BLumia requested a review from a team April 10, 2024 06:08
行为调整,当正在搜索时,回车将直接启动结果中被选中的结果(即第一个
结果),而不再是将焦点移到搜索区域。注意,当前实现仍与4/9讨论的预期
行为不同,预期是希望搜索过程中也始终显示选中项,即搜索结果区域无焦
点时,选中项也有一个近似于hover效果的指示效果的。

未在搜索时的行为不变(即,如果焦点在搜索框但没输入任何关键词就按下
回车,此时会将焦点切换到应用区域)。

Log:
@BLumia BLumia force-pushed the enter-open-1st-search-result branch from 404c521 to 34d37e5 Compare April 10, 2024 07:21
@BLumia BLumia requested a review from justforlxz April 10, 2024 07:21
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, tsic404

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@BLumia
Copy link
Member Author

BLumia commented Apr 10, 2024

/merge

@deepin-bot deepin-bot bot merged commit 0cefc23 into linuxdeepin:master Apr 10, 2024
9 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