Skip to content

feat: add more cursor sizes#118

Merged
mhduiy merged 1 commit intolinuxdeepin:masterfrom
mhduiy:cursorsize
Feb 27, 2026
Merged

feat: add more cursor sizes#118
mhduiy merged 1 commit intolinuxdeepin:masterfrom
mhduiy:cursorsize

Conversation

@mhduiy
Copy link
Contributor

@mhduiy mhduiy commented Feb 26, 2026

Updated cursor files across all three themes (bloom-dark, bloom, vintage) to include additional size variants. Modified existing cursor files and added new loginspinner cursor files for bloom-dark and bloom themes. This change provides better visual consistency and accessibility by offering cursor assets in multiple sizes to accommodate different display resolutions and user preferences.

Log: Added additional cursor size variants for improved accessibility

feat: 增加更多光标尺寸

更新了所有三个主题(bloom-dark、bloom、vintage)的光标文件以包含更
多尺寸变体。修改了现有光标文件,并为 bloom-dark 和 bloom 主题新增了
loginspinner 光标文件。此更改通过提供多种尺寸的光标资源来适应不同的显示
分辨率和用户偏好,从而提供更好的视觉一致性和可访问性。

Log: 新增光标尺寸变体以提升可访问性

PMS: BUG-343215

Updated cursor files across all three themes (bloom-dark, bloom,
vintage) to include additional size variants. Modified existing cursor
files and added new loginspinner cursor files for bloom-dark and bloom
themes. This change provides better visual consistency and accessibility
by offering cursor assets in multiple sizes to accommodate different
display resolutions and user preferences.

Log: Added additional cursor size variants for improved accessibility

feat: 增加更多光标尺寸

更新了所有三个主题(bloom-dark、bloom、vintage)的光标文件以包含更
多尺寸变体。修改了现有光标文件,并为 bloom-dark 和 bloom 主题新增了
loginspinner 光标文件。此更改通过提供多种尺寸的光标资源来适应不同的显示
分辨率和用户偏好,从而提供更好的视觉一致性和可访问性。

Log: 新增光标尺寸变体以提升可访问性

PMS: BUG-343215
@deepin-ci-robot
Copy link

deepin pr auto review

这个 Git diff 显示了一个主题项目(可能是鼠标光标主题)的变更。以下是我的审查意见:

1. 变更概述

这次提交主要涉及三个主题变体(bloom-darkbloomvintage)下的大量光标文件的更新,以及两个新文件的添加。

2. 具体变更分析

2.1 新增文件

  • bloom-dark/cursors/loginspinner (新增)
  • bloom-dark/cursors/loginspinner@2x (新增)
  • bloom/cursors/loginspinner (新增)
  • bloom/cursors/loginspinner@2x (新增)

意见

  • 新增的 loginspinner 光标及其 @2x 高分辨率版本是好的实践,这表明项目正在支持高DPI显示器。
  • 建议:确保 vintage 主题变体也添加了相应的 loginspinner 光标,以保持主题变体间的一致性。目前 diff 中未显示 vintage 主题有此文件。

2.2 二进制文件变更

  • 大量的光标文件(.png.svg.cur 等)被修改,diff 显示为 Binary files ... differ

意见

  • 由于是二进制文件,无法直接审查具体的像素或矢量变更。但从文件名来看,这是一次全面的光标样式更新。
  • 建议
    • 如果这是一次批量更新(例如,调整了所有光标的颜色或大小),建议在提交信息中说明变更的具体原因和范围。
    • 确保所有光标在三个主题变体中的样式保持一致(例如,如果 bloom-dark 中的某个光标被调整了,bloomvintage 中的对应光标也应做相应调整)。

3. 代码质量与规范

3.1 文件命名

  • 文件命名规范且一致(例如,loginspinner@2x 表示高分辨率版本)。
  • 建议:确保项目中所有高分辨率文件都遵循 @2x 的命名约定。

3.2 目录结构

  • 三个主题变体(bloom-darkbloomvintage)的目录结构清晰且一致。
  • 建议:如果未来添加更多主题变体,保持相同的目录结构。

4. 性能考虑

  • 光标文件通常是小的二进制文件,性能影响可以忽略不计。
  • 建议:如果光标文件是矢量格式(如 SVG),确保它们经过优化以减小文件大小。如果是位图格式(如 PNG),确保使用了适当的压缩。

5. 安全性

  • 二进制文件的变更通常不涉及代码注入等安全问题。
  • 建议:确保这些光标文件来自可信来源,且没有被恶意篡改。

6. 其他建议

  • 提交信息:建议在提交信息中更详细地描述这次变更的目的(例如,“更新光标样式以提升高DPI支持”或“修复光标显示问题”)。
  • 版本控制:如果可能,考虑将光标的设计源文件(如 Inkscape 的 .svg 源文件)纳入版本控制,以便于未来的修改和协作。

总结

这次提交看起来是一次全面的光标主题更新,新增了 loginspinner 光标及其高分辨率版本。整体变更合理,但建议在提交信息中提供更多背景信息,并确保三个主题变体间的一致性。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@mhduiy mhduiy merged commit 1dde222 into linuxdeepin:master Feb 27, 2026
16 checks passed
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