Skip to content

Conversation

@BLumia
Copy link
Member

@BLumia BLumia commented Mar 13, 2025

在启动器为显示状态时,直接拖拽dock边缘迅速调整dock高度会导致dock高度变回拖拽前的高度,表现行为即dock显示不完整或dock内组件使用的高度与dock自身高度不符.

根音是启动器的隐藏会触发dock隐藏状态属性变化的信号(无论是否真的变化了),此属性绑定了调整dock窗口高度的动画,动画中修改了窗口的实际高度.

由于当前仅以最保守的形式进行bug修复,此处仅添加了属性是否实际变化的检查,并仅在真的变化时才发变化的信号.此部分对应的写法应当后续改掉.

PMS: BUG-303541, BUG-303155

在启动器为显示状态时,直接拖拽dock边缘迅速调整dock高度会导致dock高度
变回拖拽前的高度,表现行为即dock显示不完整或dock内组件使用的高度与dock
自身高度不符.

根音是启动器的隐藏会触发dock隐藏状态属性变化的信号(无论是否真的变化了),
此属性绑定了调整dock窗口高度的动画,动画中修改了窗口的实际高度.

由于当前仅以最保守的形式进行bug修复,此处仅添加了属性是否实际变化的检查,
并仅在真的变化时才发变化的信号.此部分对应的写法应当后续改掉.

PMS: BUG-303541, BUG-303155
@BLumia BLumia requested review from robertkill and tsic404 March 13, 2025 06:48
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. 代码格式

    • DockPanel::init()函数中,QTimer::singleShot(10, this, [this](){...});的lambda表达式前后有多余的空格,建议删除以保持代码整洁。
  2. 逻辑改进

    • DockPanel::launcherVisibleChanged()函数中,const HideState oldHideState = hideState();const HideState newHideState = hideState();的重复调用可以合并,因为它们在同一个函数中几乎同时被调用。这样可以减少函数的复杂度,提高代码的可读性。
  3. 性能优化

    • DockPanel::launcherVisibleChanged()函数中,if (visible == m_launcherShown) return;这行代码可以放在函数开头,以避免不必要的计算。这样可以减少函数的执行时间,尤其是在hideState()函数复杂的情况下。
  4. 代码可读性

    • DockPanel::updateDockScreen()函数中,auto win = window();这行代码可以简化为auto win = window();,因为auto关键字已经足够明确地指出了变量的类型。
  5. 注释和文档

    • 虽然代码中没有明显的注释问题,但建议在关键函数和逻辑分支处添加注释,以提高代码的可读性和可维护性。

综上所述,代码在格式、逻辑、性能和可读性方面都有改进的空间。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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 c59dc9d into linuxdeepin:master Mar 13, 2025
8 of 10 checks passed
@BLumia BLumia deleted the pms-303541-303155 branch March 13, 2025 09:24
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