Closed zzxming closed 21 hours ago
This pull request introduces several enhancements to the Fluent Editor's toolbar and styling. Key changes include the addition of multiple alignment options in the toolbar configuration, the introduction of a tooltip feature for toolbar icons, and a reorganization of localization strings for better categorization. Additionally, new SCSS variables for styling management are added, and existing styles are updated to support these features. The overall functionality of the editor remains intact while expanding its capabilities, particularly in terms of user interface and internationalization.
File Path | Change Summary |
---|---|
packages/docs/fluent-editor/demos/custom-toolbar.vue |
Updated TOOLBAR_CONFIG to include multiple alignment options and reorganized list configuration. |
packages/fluent-editor/package.json |
Added new dependency: quill-toolbar-tip with version ^0.0.4 . |
packages/fluent-editor/src/assets/common.scss |
Removed $defaultFontSize , reorganized import structure, and added new configuration import. |
packages/fluent-editor/src/assets/common/config.scss |
Added new SCSS variables for font size and z-index management. |
packages/fluent-editor/src/assets/editor.scss |
Added new import for configuration, defined editor mixin, and expanded styles for various elements. |
packages/fluent-editor/src/assets/fullscreen.scss |
Updated z-index variables to use newly defined SCSS variables. |
packages/fluent-editor/src/assets/toolbar.scss |
Added tooltip styles and a new import for the Quill toolbar tip CSS. |
packages/fluent-editor/src/config/i18n/en-us.ts |
Restructured localization strings, removing and adding several keys for better organization. |
packages/fluent-editor/src/config/i18n/zh-cn.ts |
Similar restructuring of localization strings for Chinese. |
packages/fluent-editor/src/fluent-editor.ts |
Registered new ToolbarTip module and updated toolbar defaults. |
packages/fluent-editor/src/fullscreen/handler.ts |
Integrated ToolbarTip functionality into fullscreen handling. |
packages/fluent-editor/src/toolbar/index.ts |
Enhanced BetterToolbar class and added export for toolbar-tip . |
packages/fluent-editor/src/toolbar/toolbar-tip.ts |
Introduced ToolbarTip class for managing tooltip text based on language settings. |
Objective | Addressed | Explanation |
---|---|---|
Implement tooltip functionality for toolbar icons (Issue #98) | ✅ |
🐰 In the editor's realm, where tools align,
New tips and tricks, oh how they shine!
With colors and styles, all set to play,
Hover for hints, guiding the way.
A hop, a skip, through text we glide,
In this vibrant world, let creativity ride! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Hi! I'm the autofix.ci troubleshooter bot.
It looks like you correctly set up a CI job that uses the autofix.ci GitHub Action, but the autofix.ci GitHub App has not been installed for this repository. This means that autofix.ci unfortunately does not have the permissions to fix this pull request. If you are the repository owner, please install the app and then restart the CI workflow! 😃
@zzxming 已安装 autofix.ci
@kagol 这个pr使用到了包 quill-toolbar-tip
,有必要把包直接移动到 fluent-editor 里面吗
@kagol 这个pr使用到了包
quill-toolbar-tip
,有必要把包直接移动到 fluent-editor 里面吗
可以不用哈,拆分出去比较合理,现有的部分通用模块其实也应该拆分出去
PR
效果如下:
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: close #98
What is the new behavior?
Does this PR introduce a breaking change?
Other information
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
Style
Documentation