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: [workspace] drag file to weixin failed in some platform. #1910

Merged

Conversation

GongHeng2017
Copy link
Contributor

  1. If not set the mimeData text, the weixin will ignore the drag event.
  2. The weixin not override the drag event, it use the default of qt5.15.
  3. So, file manager set the text.

Log: fix issue
Bug: https://pms.uniontech.com/bug-view-248591.html

@GongHeng2017 GongHeng2017 force-pushed the 20240401-master-fixDrag branch from b14b13f to fb41e0f Compare April 1, 2024 10:02
@@ -20,6 +20,8 @@ DFMBASE_USE_NAMESPACE
DFMGLOBAL_USE_NAMESPACE
using namespace ddplugin_canvas;

inline constexpr char kCanvaMimeData[] { "CanvaMimeData" };
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

直接dde-desktop 吧

@@ -37,6 +37,8 @@ DFMGLOBAL_USE_NAMESPACE
DFMBASE_USE_NAMESPACE
using namespace dfmplugin_workspace;

inline constexpr char kFileManagerMimeData[] { "fileManagerMimeData" };
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ddefilemanager吧

@@ -22,6 +22,8 @@
DFMBASE_USE_NAMESPACE
using namespace ddplugin_organizer;

inline constexpr char kCollectionMimeData[] { "collectionMimeData" };

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ddedesktoporganizer

@GongHeng2017 GongHeng2017 force-pushed the 20240401-master-fixDrag branch 2 times, most recently from f704983 to 441e311 Compare April 2, 2024 01:12
1. If not set the mimeData text, the weixin will ignore the drag event.
2. The weixin not override the drag event, it use the default of qt5.15.
3. So, file manager set the text.

Log: fix issue
Bug: https://pms.uniontech.com/bug-view-248591.html
@GongHeng2017 GongHeng2017 force-pushed the 20240401-master-fixDrag branch from 441e311 to 11c7a9a Compare April 2, 2024 05:42
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: GongHeng2017, max-lvs

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

@GongHeng2017
Copy link
Contributor Author

/merge

@deepin-bot deepin-bot bot merged commit db8490a into linuxdeepin:master Apr 2, 2024
13 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