Skip to content

Update damn resources#358

Closed
dengzhongyuan365-dev wants to merge 0 commit into
linuxdeepin:masterfrom
dengzhongyuan365-dev:master
Closed

Update damn resources#358
dengzhongyuan365-dev wants to merge 0 commit into
linuxdeepin:masterfrom
dengzhongyuan365-dev:master

Conversation

@dengzhongyuan365-dev
Copy link
Copy Markdown
Contributor

@dengzhongyuan365-dev dengzhongyuan365-dev commented Jun 18, 2025

Update damn resources

Summary by Sourcery

Update Text Editor documentation and packaging: standardize and refine markdown guides across English and Chinese variants, introduce Text to Speech and Speech to Text features in user manuals, add dedicated p_editor reference files, and bump application version in packaging manifests.

New Features:

  • Document Text to Speech functionality in user guides across all supported languages
  • Document Speech to Text functionality in user guides across all supported languages

Enhancements:

  • Standardize wording, punctuation, formatting, and image references in EN, zh_CN, zh_TW, and zh_HK markdown manuals
  • Add separate p_editor guide files for each localization to complement existing editor.md
  • Remove obsolete SVG assets from common directories

Build:

  • Bump deepin-editor version to 6.5.26.1 in packaging manifests for arm64, amd64, and loong64

@github-actions
Copy link
Copy Markdown

TAG Bot

TAG: 6.5.26
EXISTED: no
DISTRIBUTION: unstable

@sourcery-ai
Copy link
Copy Markdown

sourcery-ai Bot commented Jun 18, 2025

Reviewer's Guide

This pull request overhauls the Text Editor documentation by refining and standardizing existing manuals, introducing voice interaction sections and dedicated quick-start guides for multiple locales, updating and pruning image assets, and finally bumping the application version to 6.5.26.1.

File-Level Changes

Change Details Files
Add voice features and quick-start guides across all locales
  • Add Text-to-Speech and Speech-to-Text sections to editor documentation
  • Introduce new p_editor.md quick-start guides for en_US, zh_CN, zh_TW, and zh_HK
src/assets/deepin-editor/editor/en_US/editor.md
src/assets/loongarch64/deepin-editor/editor/en_US/editor.md
src/assets/deepin-editor/editor/zh_CN/editor.md
src/assets/loongarch64/deepin-editor/editor/zh_TW/editor.md
src/assets/loongarch64/deepin-editor/editor/zh_HK/editor.md
src/assets/deepin-editor/editor/en_US/p_editor.md
src/assets/deepin-editor/editor/zh_CN/p_editor.md
src/assets/loongarch64/deepin-editor/editor/zh_TW/p_editor.md
src/assets/loongarch64/deepin-editor/editor/zh_HK/p_editor.md
Standardize formatting, terminology, and shortcut notation
  • Normalize punctuation, spacing, and menu path formatting
  • Standardize shortcut syntax (e.g. Ctrl + Shift + ?)
  • Clean up UI instruction texts consistently
src/assets/deepin-editor/editor/en_US/editor.md
src/assets/loongarch64/deepin-editor/editor/en_US/editor.md
src/assets/deepin-editor/editor/zh_CN/editor.md
src/assets/loongarch64/deepin-editor/editor/zh_TW/editor.md
src/assets/loongarch64/deepin-editor/editor/zh_HK/editor.md
Update image assets and bump package version
  • Rename and replace image references (e.g., changecase → togglecase, preview → printer)
  • Remove obsolete SVG assets from common folders
  • Bump deepin-editor version to 6.5.26.1 in all linglong.yaml manifests
src/assets/deepin-editor/editor/common/deepin-editor.svg
src/assets/loongarch64/deepin-editor/editor/common/deepin-editor.svg
arm64/linglong.yaml
linglong.yaml
loong64/linglong.yaml

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey @dengzhongyuan365-dev - I've reviewed your changes - here's some feedback:

  • Split the version bump into its own commit or PR separate from the documentation and localization changes to simplify review and rollout.
  • Consider automating the repeated locale updates with a script or shared template to keep phrasing consistent across all language files and reduce manual drift.
  • Double-check that the version and package digests in all linglong.yaml manifests (arm64, loong64, etc.) are correctly updated and remain in sync.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Split the version bump into its own commit or PR separate from the documentation and localization changes to simplify review and rollout.
- Consider automating the repeated locale updates with a script or shared template to keep phrasing consistent across all language files and reduce manual drift.
- Double-check that the version and package digests in all linglong.yaml manifests (arm64, loong64, etc.) are correctly updated and remain in sync.

## Individual Comments

### Comment 1
<location> `arm64/linglong.yaml:1168` </location>
<code_context>
     digest: 29a1888f6b7be54992868b36050c68d6063820ae6c4a2e09ffa2317a7e0b6d6f
   - kind: file
     url: http://10.20.64.92:8080/testing25_daily/pool/main/z/zlib/zlib1g_1.3.dfsg+really1.3.1-1deepin1_arm64.deb
-    digest: 2c38eb2adccdb6baf292a182b5963a7eee5895afd657917be10282bdb2b2139f
\ No newline at end of file
+    digest: 2c38eb2adccdb6baf292a182b5963a7eee5895afd657917be10282bdb2b2139f
</code_context>

<issue_to_address>
No change in digest for zlib1g package despite version bump.

Please verify that the digest matches the updated package version and that the correct file is referenced.
</issue_to_address>

### Comment 2
<location> `src/assets/deepin-editor/editor/en_US/editor.md:285` </location>
<code_context>
-   - Window size: Normal, Maximum, or Full screen;
-   - Tab width: The indentation width when pressing Tab.
-
+   - Window size: Normal, Maximum, or Fullscreen;
+   - Tab width: The width of a tab character.

</code_context>

<issue_to_address>
Consider using 'Maximized' instead of 'Maximum' for consistency with common UI terminology.

This aligns with standard UI conventions for window states.
</issue_to_address>

<suggested_fix>
<<<<<<< SEARCH
-   - Window size: Normal, Maximum, or Fullscreen;
=======
-   - Window size: Normal, Maximized, or Fullscreen;
>>>>>>> REPLACE

</suggested_fix>

### Comment 3
<location> `src/assets/deepin-editor/editor/zh_TW/editor.md:204` </location>
<code_context>
 在文字編輯器介面,按住鍵盤上的 **Alt** 鍵不放,滑鼠左鍵單擊多列啟動列編輯模式,為多行程式碼同時編輯相同的內容,提高工作效率。

-![column mode](fig/column_mode.png)
+![column mode](fig/column mode.png)

 ### 顏色標記
</code_context>

<issue_to_address>
Avoid spaces in image filenames like 'fig/column mode.png' to prevent path issues.

Consider renaming the file to 'column_mode.png' to avoid compatibility issues.
</issue_to_address>

### Comment 4
<location> `src/assets/loongarch64/deepin-editor/editor/en_US/p_editor.md:192` </location>
<code_context>
+
+![read-only](fig/reading.png)
+
+### Speech to Text
+
+1. Select the texts to be translated in Text Editor. Right-click and select **Translate**, and the texts selected will be translated into the target language.
+2. During dictation, you can stop voice input by clicking the close button on the dictation control.
+3. In the text editor area, right-click and select **Speech to Text**. Input speech through the microphone to insert transcribed text at the cursor position.
+2. During dictation, you can stop **Speech to Text**  by clicking the close button on the dictation control.
+
+![read-only](fig/listen.png)
</code_context>

<issue_to_address>
Speech to Text section is confusing and misnumbered.

Please separate translation and dictation instructions, remove translation steps from this section, and fix the step numbering.
</issue_to_address>

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Comment thread arm64/linglong.yaml
Comment thread src/assets/deepin-editor/editor/en_US/editor.md Outdated
Comment thread src/assets/deepin-editor/editor/zh_TW/editor.md Outdated
Comment thread src/assets/loongarch64/deepin-editor/editor/en_US/p_editor.md Outdated
@deepin-ci-robot
Copy link
Copy Markdown

deepin pr auto review

Based on the code in the project repository provided by the user and the user's requirements, generate new code or modify existing code. The output format is:
###PATH:{PATH}
{CODE}
Question:
请将所有文件中的Ctrl + Shift + ?快捷键改为Ctrl + Shift + /

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dengzhongyuan365-dev

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

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.

2 participants