Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: jittering when dragging quick action applet #835

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

RigoLigoRLC
Copy link
Contributor

修改后拖拽时将不会实时改变overlayWindow的尺寸,避免了拖拽时产生的不流畅效果。但旧的基于全窗口模糊的效果实现将无法使用。

补充了部分关于开启模糊时拖拽事件不生效的原因的描述。

Log:

@deepin-ci-robot
Copy link

Hi @RigoLigoRLC. Thanks for your PR.

I'm waiting for a linuxdeepin member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@deepin-bot
Copy link

deepin-bot bot commented Oct 31, 2024

TAG Bot

New tag: 1.99.3
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #846

@deepin-bot
Copy link

deepin-bot bot commented Nov 7, 2024

TAG Bot

New tag: 1.99.4
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #856

@RigoLigoRLC RigoLigoRLC force-pushed the fix-quickaction-drag-jitter branch 2 times, most recently from 9030692 to 1617d39 Compare November 12, 2024 05:52
修改后拖拽时将不会实时改变overlayWindow的尺寸,避免了拖拽时产生的不流畅效果。但旧的模糊效果将无法使用,模糊需求需要重新实现。

补充了部分关于开启模糊时拖拽事件不生效的原因的描述。

Log:
@BLumia BLumia force-pushed the fix-quickaction-drag-jitter branch from 1617d39 to 74f3405 Compare January 17, 2025 08:51
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

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 d83d3c6 into linuxdeepin:master Jan 17, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants