From f79dc1f91f6ee2171ecc5eab9160fb33cd582075 Mon Sep 17 00:00:00 2001 From: leo Date: Tue, 18 Jun 2024 10:13:39 +0800 Subject: [PATCH] code_review: PR #182 * avoid highlight trailing ellipsis * use WidthIncludingTrailingWhitespace instead of Width to keep original white space in translation * add same modification on zh_CN and zh_TW --- src/Resources/Locales/en_US.axaml | 32 +++++++++++----------- src/Resources/Locales/zh_CN.axaml | 38 +++++++++++++-------------- src/Resources/Locales/zh_TW.axaml | 38 +++++++++++++-------------- src/Views/NameHighlightedTextBlock.cs | 7 ++--- 4 files changed, 56 insertions(+), 59 deletions(-) diff --git a/src/Resources/Locales/en_US.axaml b/src/Resources/Locales/en_US.axaml index c67d59f9..5d110718 100644 --- a/src/Resources/Locales/en_US.axaml +++ b/src/Resources/Locales/en_US.axaml @@ -32,22 +32,22 @@ BINARY FILE NOT SUPPORTED!!! Blame BLAME ON THIS FILE IS NOT SUPPORTED!!! - Checkout${0}... + Checkout ${0}$... Compare with Branch Compare with HEAD Compare with Worktree Copy Branch Name - Delete${0}... + Delete ${0}$... Delete selected {0} branches Discard all changes - Fast-Forward to${0}$ - Git Flow - Finish${0}$ - Merge${0}$into${1}... - Pull${0}$ - Pull${0}$into${1}... - Push${0}$ - Rebase${0}$on${1}... - Rename${0}... + Fast-Forward to ${0}$ + Git Flow - Finish ${0}$ + Merge ${0}$ into ${1}$... + Pull ${0}$ + Pull ${0}$ into ${1}$... + Push ${0}$ + Rebase ${0}$ on ${1}$... + Rename ${0}$... Set Tracking Branch Unset Upstream Branch Compare @@ -85,8 +85,8 @@ Compare with HEAD Compare with Worktree Copy SHA - Rebase${0}$to Here - Reset${0}$to Here + Rebase ${0}$ to Here + Reset ${0}$ to Here Revert Commit Reword Save as Patch... @@ -139,7 +139,7 @@ Delete Branch Branch: You are about to delete a remote branch!!! - Also delete remote branch${0}$ + Also delete remote branch ${0}$ Delete Multiple Branches You are trying to delete multiple branches at one time. Be sure to double-check before taking action! Delete Remote @@ -401,7 +401,7 @@ Filter Branches LOCAL BRANCHES Navigate To HEAD - Create Branch... + Create Branch Open In {0} Open In External Tools Refresh @@ -481,8 +481,8 @@ Delete Submodule OK Copy Tag Name - Delete${0}... - Push${0}... + Delete ${0}$... + Push ${0}$... URL: Update Submodules Run `submodule update` command for this repository. diff --git a/src/Resources/Locales/zh_CN.axaml b/src/Resources/Locales/zh_CN.axaml index fc773174..9d5e7613 100644 --- a/src/Resources/Locales/zh_CN.axaml +++ b/src/Resources/Locales/zh_CN.axaml @@ -35,23 +35,23 @@ 二进制文件不支持该操作!!! 逐行追溯(blame) 选中文件不支持该操作!!! - 检出(checkout)${0}$ + 检出(checkout) ${0}$... 与其他分支对比 与当前HEAD比较 与本地工作树比较 复制分支名 - 删除${0}$ + 删除 ${0}$... 删除选中的 {0} 个分支 放弃所有更改 - 快进(fast-forward)到${0}$ - GIT工作流 - 完成${0}$ - 合并${0}$到${1}$ - 拉回(pull)${0}$ - 拉回(pull)${0}$内容至${1}$ + 快进(fast-forward)到 ${0}$ + GIT工作流 - 完成 ${0}$ + 合并 ${0}$ 到 ${1}$... + 拉回(pull) ${0}$ + 拉回(pull) ${0}$ 内容至 ${1}$... 推送(push)${0}$ - 变基(rebase)${0}$分支至${1}$ - 重命名${0}$ - 切换上游分支... + 变基(rebase) ${0}$ 分支至 ${1}$... + 重命名 ${0}$... + 切换上游分支 取消追踪 分支比较 字节 @@ -88,8 +88,8 @@ 与当前HEAD比较 与本地工作树比较 复制提交指纹 - 变基(rebase)${0}$到此处 - 重置(reset)${0}$到此处 + 变基(rebase) ${0}$ 到此处 + 重置(reset) ${0}$ 到此处 回滚此提交 编辑提交信息 另存为补丁 ... @@ -142,7 +142,7 @@ 删除分支确认 分支名 : 您正在删除远程上的分支,请务必小心!!! - 同时删除远程分支${0}$ + 同时删除远程分支 ${0}$ 删除多个分支 您正在尝试一次性删除多个分支,请务必仔细检查后再执行操作! 删除远程确认 @@ -196,8 +196,8 @@ 放弃选中的更改 使用外部合并工具打开 另存为补丁... - 暂存(add)... - 暂存(add){0} 个文件... + 暂存(add) + 暂存(add){0} 个文件 暂存选中的更改 贮藏(stash)... 贮藏(stash)选中的 {0} 个文件... @@ -387,9 +387,9 @@ 复制远程地址 删除 ... 编辑 ... - 拉取(fetch)更新 ... + 拉取(fetch)更新 在浏览器中打开 - 清理远程已删除分支 ... + 清理远程已删除分支 分支重命名 新的名称 : 新的分支名不能与现有分支名相同 @@ -483,8 +483,8 @@ 删除子模块 确 定 复制标签名 - 删除${0}$ - 推送${0}$ + 删除 ${0}$... + 推送 ${0}$... 仓库地址 : 更新子模块 为此仓库执行`submodule update`命令,更新所有的子模块。 diff --git a/src/Resources/Locales/zh_TW.axaml b/src/Resources/Locales/zh_TW.axaml index ef50a35e..0611cbf9 100644 --- a/src/Resources/Locales/zh_TW.axaml +++ b/src/Resources/Locales/zh_TW.axaml @@ -35,23 +35,23 @@ 二進位制檔案不支援該操作!!! 逐行追溯(blame) 選中檔案不支援該操作!!! - 檢出(checkout)${0}$ + 檢出(checkout) ${0}$... 與其他分支比較 與當前HEAD比較 與本地工作樹比較 複製分支名 - 刪除${0}$ + 刪除 ${0}$... 刪除選中的 {0} 個分支 放棄所有更改 - 快進(fast-forward)到${0}$ - GIT工作流 - 完成${0}$ - 合併${0}$到${1}$ - 拉回(pull)${0}$ - 拉回(pull)${0}$內容至${1}$ + 快進(fast-forward)到 ${0}$ + GIT工作流 - 完成 ${0}$ + 合併 ${0}$ 到 ${1}$... + 拉回(pull) ${0}$ + 拉回(pull) ${0}$ 內容至 ${1}$... 推送(push)${0}$ - 變基(rebase)${0}$分支至${1}$ - 重新命名${0}$ - 切換上游分支... + 變基(rebase) ${0}$ 分支至 ${1}$... + 重新命名 ${0}$... + 切換上游分支 取消追蹤 分支比較 位元組 @@ -88,8 +88,8 @@ 與當前HEAD比較 與本地工作樹比較 複製提交指紋 - 變基(rebase)${0}$到此處 - 重置(reset)${0}$到此處 + 變基(rebase) ${0}$ 到此處 + 重置(reset) ${0}$ 到此處 回滾此提交 編輯提交資訊 另存為補丁 ... @@ -142,7 +142,7 @@ 刪除分支確認 分支名 : 您正在刪除遠端上的分支,請務必小心!!! - 同時刪除遠端分支${0}$ + 同時刪除遠端分支 ${0}$ 刪除多個分支 您正在嘗試一次性刪除多個分支,請務必仔細檢查後再執行操作! 刪除遠端確認 @@ -196,8 +196,8 @@ 放棄選中的更改 使用外部合併工具開啟 另存為補丁... - 暫存(add)... - 暫存(add){0} 個檔案... + 暫存(add) + 暫存(add){0} 個檔案 暫存選中的更改 儲藏(stash)... 儲藏(stash)選中的 {0} 個檔案... @@ -387,9 +387,9 @@ 複製遠端地址 刪除 ... 編輯 ... - 拉取(fetch)更新 ... + 拉取(fetch)更新 在瀏覽器中訪問網址 - 清理遠端已刪除分支 ... + 清理遠端已刪除分支 分支重新命名 新的名稱 : 新的分支名不能與現有分支名相同 @@ -483,8 +483,8 @@ 刪除子模組 確 定 複製標籤名 - 刪除${0}$ - 推送${0}$ + 刪除 ${0}$... + 推送 ${0}$... 倉庫地址 : 更新子模組 本操作將執行 `submodule update` 。 diff --git a/src/Views/NameHighlightedTextBlock.cs b/src/Views/NameHighlightedTextBlock.cs index 3be81186..c0583223 100644 --- a/src/Views/NameHighlightedTextBlock.cs +++ b/src/Views/NameHighlightedTextBlock.cs @@ -81,7 +81,6 @@ namespace SourceGit.Views return; var normalTypeface = new Typeface(FontFamily, FontStyle.Normal, FontWeight.Normal, FontStretch.Normal); - //var highlightTypeface = new Typeface(FontFamily, FontStyle.Normal, FontWeight.Bold, FontStretch.Normal); var underlinePen = new Pen(Foreground, 1); var offsetX = 0.0; @@ -99,7 +98,6 @@ namespace SourceGit.Views part, CultureInfo.CurrentCulture, FlowDirection.LeftToRight, - //isName ? highlightTypeface : normalTypeface, normalTypeface, FontSize, Foreground); @@ -107,15 +105,14 @@ namespace SourceGit.Views if (isName) { var lineY = formatted.Baseline + 2; - offsetX += 4; context.DrawText(formatted, new Point(offsetX, 0)); context.DrawLine(underlinePen, new Point(offsetX, lineY), new Point(offsetX + formatted.Width, lineY)); - offsetX += formatted.Width + 4; + offsetX += formatted.WidthIncludingTrailingWhitespace; } else { context.DrawText(formatted, new Point(offsetX, 0)); - offsetX += formatted.Width; + offsetX += formatted.WidthIncludingTrailingWhitespace; } isName = !isName;