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: provide find and replace functionality for the default rich text editor #5206

Merged
merged 6 commits into from
Jan 24, 2024

Conversation

LIlGG
Copy link
Member

@LIlGG LIlGG commented Jan 19, 2024

What type of PR is this?

/kind feature

What this PR does / why we need it:

为默认富文本编辑器添加查找与搜索的功能扩展。

快捷键:

当焦点处于编辑器中时,可以使用 Mod+f 来唤起查找与搜索框,或者点击顶部工具栏来打开。
当焦点处于查找与搜索框时,按下 Ecs 可进行关闭。

image

How to test it?

测试查找与搜索功能是否正常

Which issue(s) this PR fixes:

Fixes #5195

Does this PR introduce a user-facing change?

为默认富文本编辑器添加查找与搜索的功能扩展。

@f2c-ci-robot f2c-ci-robot bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/feature Categorizes issue or PR as related to a new feature. labels Jan 19, 2024
@CLAassistant
Copy link

CLAassistant commented Jan 19, 2024

CLA assistant check
All committers have signed the CLA.

@f2c-ci-robot f2c-ci-robot bot requested review from QuentinHsu and wzrove January 19, 2024 03:45
@LIlGG
Copy link
Member Author

LIlGG commented Jan 19, 2024

/area console
/area editor
/milestone 2.12.x

@f2c-ci-robot f2c-ci-robot bot added this to the 2.12.x milestone Jan 19, 2024
@f2c-ci-robot f2c-ci-robot bot added area/console Issues or PRs related to the Halo Console area/editor Issues or PRs related to the Editor labels Jan 19, 2024
@ruibaby
Copy link
Member

ruibaby commented Jan 19, 2024

图片

红框中的搜索高亮似乎有问题。

@LIlGG LIlGG force-pushed the feat/editor-search-and-replace branch from a59d8a0 to 147fe86 Compare January 19, 2024 06:02
@LIlGG
Copy link
Member Author

LIlGG commented Jan 19, 2024

红框中的搜索高亮似乎有问题。

已解决。

Copy link

codecov bot commented Jan 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9d9b152) 57.05% compared to head (9146208) 57.05%.
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #5206   +/-   ##
=========================================
  Coverage     57.05%   57.05%           
  Complexity     3332     3332           
=========================================
  Files           584      584           
  Lines         19189    19189           
  Branches       1444     1444           
=========================================
  Hits          10949    10949           
  Misses         7664     7664           
  Partials        576      576           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ruibaby
Copy link
Member

ruibaby commented Jan 19, 2024

图片

当输入框文字过长,会有显示问题。

@LIlGG
Copy link
Member Author

LIlGG commented Jan 19, 2024

当输入框文字过长,会有显示问题。

已解决

@f2c-ci-robot f2c-ci-robot bot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jan 19, 2024
@LIlGG LIlGG force-pushed the feat/editor-search-and-replace branch from 140b7e9 to 683b8e3 Compare January 22, 2024 02:25
Signed-off-by: Ryan Wang <i@ryanc.cc>
@LIlGG LIlGG requested a review from ruibaby January 22, 2024 08:15
Copy link
Member

@ruibaby ruibaby left a comment

Choose a reason for hiding this comment

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

/lgtm

@f2c-ci-robot f2c-ci-robot bot added the lgtm Indicates that a PR is ready to be merged. label Jan 24, 2024
@LIlGG LIlGG requested a review from ruibaby January 24, 2024 06:39
Copy link
Member

@JohnNiang JohnNiang left a comment

Choose a reason for hiding this comment

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

/approve

@f2c-ci-robot f2c-ci-robot bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 24, 2024
Copy link
Member

@ruibaby ruibaby left a comment

Choose a reason for hiding this comment

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

/approve

Copy link

f2c-ci-robot bot commented Jan 24, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: guqing, JohnNiang, ruibaby

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

The pull request process is described 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

@f2c-ci-robot f2c-ci-robot bot merged commit 3846525 into halo-dev:main Jan 24, 2024
7 checks passed
@ruibaby ruibaby modified the milestones: 2.12.x, 2.12.0 Jan 25, 2024
@LIlGG LIlGG deleted the feat/editor-search-and-replace branch February 1, 2024 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/console Issues or PRs related to the Halo Console area/editor Issues or PRs related to the Editor kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

期望默认编辑器支持搜索和替换文字
5 participants