Skip to content

Conversation

@robertkill
Copy link
Contributor

@robertkill robertkill commented Jun 11, 2025

Improve tray item visibility handling during updates

  • Add isUpdating flag to TraySortOrderModel to track visual index updates
  • Hide tray items during model updates to prevent visual glitches
  • Only show drop hover indicators during active drag operations
  • Add itemVisible checks to delegate visibility conditions

This ensures smoother visual transitions when the tray model is being updated and prevents artifacts during drag operations.

pms: BUG-289521

Summary by Sourcery

Prevent tray icon flashing by hiding items during visual index updates and restricting hover indicators to active drags

Bug Fixes:

  • Hide tray items during model updates to avoid flickering artifacts
  • Only display drop hover indicators when a drag operation is ongoing

Enhancements:

  • Add isUpdating flag to TraySortOrderModel to signal visual index update state
  • Introduce itemVisible checks in QML delegates to manage tray item visibility during updates

Improve tray item visibility handling during updates

- Add `isUpdating` flag to TraySortOrderModel to track visual index updates
- Hide tray items during model updates to prevent visual glitches
- Only show drop hover indicators during active drag operations
- Add `itemVisible` checks to delegate visibility conditions

This ensures smoother visual transitions when the tray model is being updated and prevents artifacts during drag operations.

pms: BUG-289521
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@robertkill
Copy link
Contributor Author

/forcemerge

@deepin-bot
Copy link

deepin-bot bot commented Jun 11, 2025

This pr force merged! (status: blocked)

@deepin-bot deepin-bot bot merged commit f153a59 into linuxdeepin:master Jun 11, 2025
7 of 10 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