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

chore: bump default editor version to 27 #4288

Merged
merged 5 commits into from Jul 24, 2023

Conversation

ruibaby
Copy link
Member

@ruibaby ruibaby commented Jul 24, 2023

What type of PR is this?

/area console
/area editor
/milestone 2.8.x
/kind feature

What this PR does / why we need it:

升级默认编辑器以支持:

  1. 重构 Block 操作区域逻辑,防止误操作。 Fixes 编辑器的图片编辑框杂乱,建议浏览到下一张图片时就取消上个文件的编辑框 #4164
  2. 支持设置文字背景颜色和字体颜色。
  3. iframe block 支持主动刷新页面。 编辑器嵌入网页功能需要支持主动刷新 #4227
  4. 在顶部工具栏的区域添加工具箱。
  5. 支持设置字体大小。

Does this PR introduce a user-facing change?

升级默认编辑器版本,支持设置字体颜色、大小等功能,优化使用体验。

Signed-off-by: Ryan Wang <i@ryanc.cc>
@f2c-ci-robot f2c-ci-robot bot added area/console Issues or PRs related to the Halo Console release-note Denotes a PR that will be considered when it comes time to generate release notes. labels Jul 24, 2023
@f2c-ci-robot f2c-ci-robot bot added this to the 2.8.x milestone Jul 24, 2023
@f2c-ci-robot f2c-ci-robot bot added area/editor Issues or PRs related to the Editor kind/feature Categorizes issue or PR as related to a new feature. labels Jul 24, 2023
@f2c-ci-robot f2c-ci-robot bot requested review from wan92hen and wzrove July 24, 2023 09:02
@codecov
Copy link

codecov bot commented Jul 24, 2023

Codecov Report

Merging #4288 (c4bbff8) into main (6fa442d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #4288   +/-   ##
=========================================
  Coverage     60.62%   60.62%           
  Complexity     2498     2498           
=========================================
  Files           370      370           
  Lines         13047    13047           
  Branches        935      935           
=========================================
  Hits           7910     7910           
  Misses         4678     4678           
  Partials        459      459           

Copy link
Member

@guqing guqing 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 Jul 24, 2023
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.

需要解决一下 CI 错误:

> @halo-dev/console@2.7.0 typecheck /home/runner/work/halo/halo/console
> vue-tsc --noEmit -p tsconfig.app.json --composite false && pnpm run typecheck:packages

Error: src/components/editor/DefaultEditor.vue(470,5): error TS2322: Type 'string' is not assignable to type '"en" | "en-US" | "zh-CN" | "zh" | undefined'.
 ELIFECYCLE  Command failed with exit code 2.
make: *** [Makefile:14: lint] Error 1
make: Leaving directory '/home/runner/work/halo/halo/console'
Error: Process completed with exit code 2.

See https://github.com/halo-dev/halo/actions/runs/5642892304/job/15283662005?pr=4288 for more.

Signed-off-by: Ryan Wang <i@ryanc.cc>
@f2c-ci-robot f2c-ci-robot bot removed the lgtm Indicates that a PR is ready to be merged. label Jul 24, 2023
Signed-off-by: Ryan Wang <i@ryanc.cc>
@ruibaby
Copy link
Member Author

ruibaby commented Jul 24, 2023

/hold

此版本的内容区域无法滚动。

@f2c-ci-robot f2c-ci-robot bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 24, 2023
Signed-off-by: Ryan Wang <i@ryanc.cc>
@ruibaby ruibaby removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 24, 2023
@ruibaby
Copy link
Member Author

ruibaby commented Jul 24, 2023

@halo-dev/sig-halo-console

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.

/unhold
/lgtn

@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Jul 24, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JohnNiang

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 added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 24, 2023
@JohnNiang
Copy link
Member

/lgtm

@f2c-ci-robot f2c-ci-robot bot added the lgtm Indicates that a PR is ready to be merged. label Jul 24, 2023
@f2c-ci-robot f2c-ci-robot bot merged commit f56de04 into halo-dev:main Jul 24, 2023
4 checks passed
@ruibaby ruibaby deleted the chore/bump-editor-version-27 branch July 24, 2023 15:00
@ruibaby ruibaby modified the milestones: 2.8.x, 2.8.0 Jul 26, 2023
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.

编辑器的图片编辑框杂乱,建议浏览到下一张图片时就取消上个文件的编辑框
3 participants