fix: adjust input field placeholder text color

[18202781743]

1. Changed placeholder text opacity from 0.7 to 0.4 in FlowStyle.qml for
   better visual hierarchy
2. Moved placeholderText palette definition from FlowStyle to searchEdit
   section for proper scoping
3. Added placeholderTextPalette property binding in SearchEdit.qml to
   use the new color settings
4. This adjustment improves readability and visual distinction between
   placeholder text and user input

Log: Adjusted placeholder text color in search input fields for better
visual clarity

Influence:

1. Test search input fields to verify placeholder text appears with
   reduced opacity (0.4)
2. Verify placeholder text is visible but clearly distinguishable from
   user-entered text
3. Check consistency across light and dark themes
4. Test in various input states (focused, unfocused, with text, without
   text)
5. Ensure accessibility requirements are still met with the new color
   contrast

fix: 调整输入框占位符文本颜色

1. 将 FlowStyle.qml 中的占位符文本不透明度从 0.7 调整为 0.4，以改善视觉
   层次
2. 将 placeholderText 调色板定义从 FlowStyle 移动到 searchEdit 部分，实
   现正确的作用域
3. 在 SearchEdit.qml 中添加 placeholderTextPalette 属性绑定以使用新的颜
   色设置
4. 此调整提高了可读性，并改善了占位符文本与用户输入之间的视觉区分

Log: 调整搜索输入框占位符文本颜色，提升视觉清晰度

Influence:

1. 测试搜索输入框，验证占位符文本以降低的不透明度（0.4）显示
2. 验证占位符文本可见但与用户输入文本有明显区分
3. 检查浅色和深色主题下的一致性
4. 测试各种输入状态（聚焦、未聚焦、有文本、无文本）
5. 确保新的颜色对比度仍满足无障碍访问要求

PMS: BUG-338425

[sourcery-ai]

Sorry @18202781743, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

[deepin-ci-robot]

deepin pr auto review

这段代码的 diff 主要涉及两个 QML 文件：FlowStyle.qml 和 SearchEdit.qml。改动目的是为了调整搜索框（SearchEdit）中占位符文本的颜色，并为其应用了特定的样式。

以下是对这段 diff 的详细审查意见，包括语法逻辑、代码质量、代码性能和代码安全方面的分析：

1. 语法逻辑

• 正确性：代码在语法上是正确的。QML 的属性绑定和对象定义结构符合规范。
• 逻辑一致性：
  • 在 FlowStyle.qml 中，searchEdit 对象的 placeholderText 颜色透明度从 0.7 降低到了 0.4。这使得占位符文本看起来更淡，通常这符合 UI 设计规范，以便与用户实际输入的文本（通常是不透明度较高的黑色或白色）区分开来。
  • 在 FlowStyle.qml 中，新增了一个 searchInput 对象，其 placeholderText 透明度保持为 0.7。这表明代码中可能存在两种不同的搜索框样式：一种是 searchEdit，另一种是 searchInput。
  • 在 SearchEdit.qml 中，通过 placeholderTextPalette: DS.Style.searchEdit.placeholderText 显式引用了样式定义。这确保了 UI 组件使用的是 FlowStyle.qml 中定义的最新样式（即透明度为 0.4 的那个）。

2. 代码质量

• 命名规范：变量名如 placeholderText, normal, normalDark 符合 QML 和 DTK 的命名习惯，清晰易懂。
• 代码复用与一致性：
  • 潜在问题：在 FlowStyle.qml 中，searchEdit 和 searchInput 都定义了 placeholderText，但透明度值不同（0.4 vs 0.7）。如果这两个组件在视觉上应该表现一致，那么这里就存在不一致性。
  • 建议：请确认 searchEdit 和 searchInput 是否代表两种不同视觉状态的搜索框。如果它们应该是统一的，建议统一透明度值，或者提取一个公共的 placeholderText 常量供两者使用，避免维护两套数值。
  • 结构：在 SearchEdit.qml 中，显式绑定 placeholderTextPalette 是好的做法，它使得样式来源清晰，便于后续维护。

3. 代码性能

• 性能影响：此改动主要涉及静态属性的赋值和简单的颜色计算（Qt.rgba）。这不会对运行时性能产生负面影响。
• 对象创建：D.Palette 对象的创建是在组件加载时进行的，不是在频繁调用的热路径中，因此不会造成性能问题。

4. 代码安全

• 安全性：此代码仅涉及 UI 样式的调整，不涉及文件操作、网络请求或用户输入处理，因此没有安全风险。
• 健壮性：代码依赖于 DS.Style.searchEdit.placeholderText 的存在。只要 FlowStyle.qml 编译通过且结构正确，运行时就不会出现空指针或属性未定义的错误。

总结与改进建议

这段代码的修改逻辑清晰，目的是为了优化搜索框占位符的视觉层次感。代码本身没有语法错误或性能隐患。

改进建议：

1. 统一样式定义：
   如前所述，FlowStyle.qml 中同时存在 searchEdit 和 searchInput 两个对象，且它们的 placeholderText 透明度不一致。

   • 如果这是有意为之（例如一个是主搜索框，一个是输入框），建议在代码注释中说明两者的区别，以免后续开发者混淆。
   • 如果这是无意造成的重复，建议合并这两个定义，或者确保它们的属性值保持一致。

2. 使用常量或枚举：
   如果透明度 0.4 是一个设计规范中的标准值（例如 "Secondary Text" 或 "Placeholder"），建议在全局样式文件中定义一个常量，而不是硬编码 0.4。这样如果设计规范变更，只需修改一处。

   • 例如：定义 readonly property real placeholderOpacity: 0.4，然后引用它。

3. 代码注释：
   在 SearchEdit.qml 中添加一行注释说明为什么需要显式绑定 placeholderTextPalette，或者为什么它默认没有继承（如果有特殊原因），有助于代码的可读性。

   // 显式使用 searchEdit 的占位符样式，确保与其他搜索框视觉一致
   placeholderTextPalette: DS.Style.searchEdit.placeholderText

示例代码优化（针对 FlowStyle.qml）：

// 建议在 FlowStyle 顶部定义通用透明度常量
readonly property real placeholderOpacity: 0.4 
// 或者如果 searchInput 确实需要不同的透明度
readonly property real inputPlaceholderOpacity: 0.7

QtObject {
    id: searchEdit
    // ...
    property D.Palette placeholderText: D.Palette {
        normal: Qt.rgba(0, 0, 0, placeholderOpacity)
        normalDark: Qt.rgba(1, 1, 1, placeholderOpacity)
    }
}

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, mhduiy

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment