Skip to content

chore: update copywriting of dock modes#1118

Merged
BLumia merged 1 commit into
linuxdeepin:masterfrom
BLumia:pms-312611
May 8, 2025
Merged

chore: update copywriting of dock modes#1118
BLumia merged 1 commit into
linuxdeepin:masterfrom
BLumia:pms-312611

Conversation

@BLumia
Copy link
Copy Markdown
Member

@BLumia BLumia commented May 8, 2025

根据产品文档,调整dock两种模式的显示名称文案,并与控制中心的文案对齐.

Summary by Sourcery

Update copywriting for dock modes to align with product documentation and control center terminology

New Features:

  • Added two new dock mode labels: 'Classic Mode' and 'Centered Mode'

Chores:

  • Updated translation files to reflect new dock mode terminology

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 @BLumia - I've reviewed your changes - here's some feedback:

  • Many translations for the new 'Classic Mode' and 'Centered Mode' dock options are unfinished.
  • The removal of the 'All Windows' string from task manager items appears unrelated to updating dock mode copywriting and should be clarified.
Here's what I looked at during the review
  • 🟢 General issues: all looks good
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

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.

根据产品文档,调整dock两种模式的显示名称文案,并与控制中心的文案对齐.

额外的,现在翻译不再附带行数/代码位置信息.

PMS: BUG-312611
Log:
@sourcery-ai
Copy link
Copy Markdown

sourcery-ai Bot commented May 8, 2025

Reviewer's Guide

This pull request updates the display names for dock modes to "Classic Mode" and "Centered Mode". This was implemented by modifying source strings in panels/dock/package/main.qml and updating their translations in panels/dock/translations/org.deepin.ds.dock_*.ts files. These dock translation files, along with numerous other translation files across various UI components, also had their XML declarations standardized to encoding="utf-8" and internal line number references adjusted. Additionally, a specific string ("All Windows") was removed from Chinese translations for the task manager.

File-Level Changes

Change Details Files
Renamed dock modes to "Classic Mode" and "Centered Mode" in QML and updated corresponding translations.
  • Modified QML to use "Classic Mode" and "Centered Mode" for dock item alignment options.
  • Updated and added translations for the new mode names.
  • Removed or marked as "vanished" old alignment-based mode names (e.g., "Align Left", "Align Top", "Align Center").
  • Standardized XML declaration to include encoding="utf-8" and updated line numbers in these dock translation files.
panels/dock/package/main.qml
panels/dock/translations/org.deepin.ds.dock_ru.ts
panels/dock/translations/org.deepin.ds.dock_az.ts
panels/dock/translations/org.deepin.ds.dock_ca.ts
panels/dock/translations/org.deepin.ds.dock_es.ts
panels/dock/translations/org.deepin.ds.dock_fi.ts
panels/dock/translations/org.deepin.ds.dock_fr.ts
panels/dock/translations/org.deepin.ds.dock_hu.ts
panels/dock/translations/org.deepin.ds.dock_pl.ts
panels/dock/translations/org.deepin.ds.dock_pt_BR.ts
panels/dock/translations/org.deepin.ds.dock_uk.ts
panels/dock/translations/org.deepin.ds.dock_zh_CN.ts
panels/dock/translations/org.deepin.ds.dock_zh_HK.ts
panels/dock/translations/org.deepin.ds.dock_zh_TW.ts
panels/dock/translations/org.deepin.ds.dock.ts
panels/dock/translations/org.deepin.ds.dock_bo.ts
panels/dock/translations/org.deepin.ds.dock_it.ts
panels/dock/translations/org.deepin.ds.dock_ja.ts
panels/dock/translations/org.deepin.ds.dock_ko.ts
panels/dock/translations/org.deepin.ds.dock_nb_NO.ts
Applied encoding="utf-8" to XML declarations and updated line number references in numerous non-dock-specific translation files.
  • Modified the XML declaration at the beginning of translation files to include encoding="utf-8".
  • Updated line numbers in <location> tags.
panels/notification/osd/default/translations/org.deepin.ds.osd.default_az.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_ca.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_es.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_fi.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_fr.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_hu.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_ja.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_pl.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_pt_BR.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_ru.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_uk.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_zh_CN.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_zh_HK.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_zh_TW.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_bo.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_it.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_ko.ts
panels/notification/osd/default/translations/org.deepin.ds.osd.default_nb_NO.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_ru.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_az.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_ca.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_es.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_fi.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_fr.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_hu.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_ja.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_pl.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_pt_BR.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_uk.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_zh_CN.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_zh_HK.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_zh_TW.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_bo.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_it.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_ko.ts
panels/notification/center/translations/org.deepin.ds.notificationcenter_nb_NO.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_ru.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_az.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_ca.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_es.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_fi.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_fr.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_hu.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_ja.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_pl.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_pt_BR.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_uk.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_zh_CN.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_zh_HK.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_zh_TW.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_bo.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_it.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_ko.ts
panels/notification/bubble/translations/org.deepin.ds.notificationbubble_nb_NO.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_az.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_bo.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_ca.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_es.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_fi.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_fr.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_hu.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_it.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_ja.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_ko.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_nb_NO.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_pl.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_pt_BR.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_ru.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_uk.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_az.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_bo.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_ca.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_es.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_fi.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_fr.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_hu.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_it.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_ja.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_ko.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_nb_NO.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_pl.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_pt_BR.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_ru.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_uk.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_zh_CN.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_zh_HK.ts
panels/notification/osd/windoweffect/translations/org.deepin.ds.osd.windoweffect_zh_TW.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_az.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_ca.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_es.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_fi.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_fr.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_hu.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_ja.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_pl.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_pt_BR.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_ru.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_uk.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_zh_CN.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_zh_HK.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_zh_TW.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_bo.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_it.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_ko.ts
panels/notification/osd/displaymode/translations/org.deepin.ds.osd.displaymode_nb_NO.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_az.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_bo.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_ca.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_es.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_fi.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_fr.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_hu.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_it.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_ja.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_ko.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_nb_NO.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_pl.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_pt_BR.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_ru.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_uk.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_zh_HK.ts
panels/dock/multitaskview/translations/org.deepin.ds.dock.multitaskview_zh_TW.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_az.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_bo.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_ca.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_es.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_fi.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_fr.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_hu.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_it.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_ja.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_ko.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_nb_NO.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_pl.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_pt_BR.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_ru.ts
panels/dock/showdesktop/translations/org.deepin.ds.dock.showdesktop_uk.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_az.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_bo.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_ca.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_es.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_fi.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_fr.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_hu.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_it.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_ja.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_ko.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_nb_NO.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_pl.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_pt_BR.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_ru.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_uk.ts
panels/dock/tray/translations/org.deepin.ds.dock.tray_zh_CN.ts
Marked the "All Windows" string as vanished in Chinese localizations for the task manager.
  • Set translation type to "vanished" for the "All Windows" source string.
  • These files also received XML encoding and line number updates.
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_zh_CN.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_zh_HK.ts
panels/dock/taskmanager/translations/org.deepin.ds.dock.taskmanager_zh_TW.ts

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

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, yixinshark

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

@BLumia BLumia merged commit 08464c6 into linuxdeepin:master May 8, 2025
7 of 10 checks passed
@BLumia BLumia deleted the pms-312611 branch May 8, 2025 11:17
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