Skip to content

fix: Wrong case sensitive config#281

Merged
rb-union merged 1 commit intolinuxdeepin:masterfrom
rb-union:master
Jul 3, 2024
Merged

fix: Wrong case sensitive config#281
rb-union merged 1 commit intolinuxdeepin:masterfrom
rb-union:master

Conversation

@rb-union
Copy link
Contributor

@rb-union rb-union commented Jul 3, 2024

Change the setting when case sensitive.
调整区分大小写配置时的设置

Log: 修复错误的匹配大小写判断设置
Bug: https://pms.uniontech.com/bug-view-262733.html

Change the setting when case sensitive.
调整区分大小写配置时的设置

Log: 修复错误的匹配大小写判断设置
Bug: https://pms.uniontech.com/bug-view-262733.html
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • findKeywordForwardsearchKeywordSelection函数中,options变量被错误地修改为与defaultCaseSensitive变量的值进行按位或操作,而不是按位与操作。这会导致options变量在后续的find函数调用中始终包含QTextDocument::FindCaseSensitively标志,即使defaultCaseSensitiveQt::CaseInsensitive

是否建议立即修改:

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: pengfeixx, rb-union

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

Details 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

@rb-union rb-union merged commit e786bdd into linuxdeepin:master Jul 3, 2024
rb-union added a commit to rb-union/deepin-editor that referenced this pull request Jul 3, 2024
Bump version to 6.0.18
PR:
* linuxdeepin#280
* linuxdeepin#281

Log: Bump version to 6.0.18
rb-union added a commit that referenced this pull request Jul 4, 2024
Bump version to 6.0.18
PR:
* #280
* #281

Log: Bump version to 6.0.18
rb-union added a commit to rb-union/deepin-editor that referenced this pull request Aug 20, 2024
Bump version to 6.0.18
PR:
* linuxdeepin#280
* linuxdeepin#281

Log: Bump version to 6.0.18
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