Skip to content

fix: The tray application flashes an icon when opened#1150

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
robertkill:master
Jun 11, 2025
Merged

fix: The tray application flashes an icon when opened#1150
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
robertkill:master

Conversation

@robertkill

@robertkill robertkill commented Jun 11, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown

[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
Copy Markdown
Contributor Author

/forcemerge

@deepin-bot

deepin-bot Bot commented Jun 11, 2025

Copy link
Copy Markdown

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