mirror of
https://github.com/EGroupware/egroupware.git
synced 2025-01-21 21:38:55 +01:00
Replace the drag out trigger key from Ctrl (or Command key in Mac) to Shift key.
- Make Ctrl key as a standard dnd action disabling, and being able to select content by holding Ctrl key.
This commit is contained in:
parent
5391fd1674
commit
c1c4b7c682
@ -132,12 +132,11 @@ function egwDragActionImplementation()
|
|||||||
var text = $j(document.createElement('div')).addClass('et2_egw_action_ddHelper_tip');
|
var text = $j(document.createElement('div')).addClass('et2_egw_action_ddHelper_tip');
|
||||||
div.append(text);
|
div.append(text);
|
||||||
|
|
||||||
// Add notice of Ctrl key, if supported
|
// Add notice of Shift key, if supported
|
||||||
if('draggable' in document.createElement('span') &&
|
if('draggable' in document.createElement('span') &&
|
||||||
navigator && navigator.userAgent.indexOf('Chrome') >= 0)
|
navigator && navigator.userAgent.indexOf('Chrome') >= 0)
|
||||||
{
|
{
|
||||||
var key = ["Mac68K","MacPPC","MacIntel"].indexOf(window.navigator.platform) < 0 ? 'Ctrl' : 'Command';
|
text.text(egw.lang('Hold Shift to drag %1 to your computer', itemLabel));
|
||||||
text.text(egw.lang('Hold %1 to drag %2 to your computer',key, itemLabel));
|
|
||||||
}
|
}
|
||||||
// Final html DOM return as helper structor
|
// Final html DOM return as helper structor
|
||||||
return div;
|
return div;
|
||||||
@ -174,15 +173,19 @@ function egwDragActionImplementation()
|
|||||||
}
|
}
|
||||||
if(action)
|
if(action)
|
||||||
{
|
{
|
||||||
|
// As we are planning to remove this trigger key for drag out action in the future, and as
|
||||||
|
// we have a standard key (ctrl or command key in Mac) in nm context menu to be able to
|
||||||
|
// disable the context menu, therefore, we will make the Ctrl key a standard key to disabling dd actions too,
|
||||||
|
// which means user also would be able to select content by holding ctrl key.
|
||||||
/**
|
/**
|
||||||
* We found an action with dragType 'file', so by holding Ctrl
|
* We found an action with dragType 'file', so by holding Shift
|
||||||
* key & dragging, user can drag from browser to system.
|
* key & dragging, user can drag from browser to system.
|
||||||
* The global data store must provide a full, absolute URL in 'download_url'
|
* The global data store must provide a full, absolute URL in 'download_url'
|
||||||
* and a mime in 'mime'.
|
* and a mime in 'mime'.
|
||||||
*
|
*
|
||||||
* Unfortunately, Native DnD to drag the file conflicts with jQueryUI draggable,
|
* Unfortunately, Native DnD to drag the file conflicts with jQueryUI draggable,
|
||||||
* which handles all the other DnD actions. We get around this by:
|
* which handles all the other DnD actions. We get around this by:
|
||||||
* 1. Require the user indicate a file drag with Ctrl key
|
* 1. Require the user indicate a file drag with Shift key
|
||||||
* 2. Disable jQueryUI draggable, then turn on native draggable attribute
|
* 2. Disable jQueryUI draggable, then turn on native draggable attribute
|
||||||
* This way we can at least toggle which one is operating, so they
|
* This way we can at least toggle which one is operating, so they
|
||||||
* both work alternately if not together.
|
* both work alternately if not together.
|
||||||
@ -193,12 +196,12 @@ function egwDragActionImplementation()
|
|||||||
|
|
||||||
$j(node).off("mousedown")
|
$j(node).off("mousedown")
|
||||||
.on("mousedown", function(event) {
|
.on("mousedown", function(event) {
|
||||||
$j(node).draggable("option","disabled",event.ctrlKey || event.metaKey);
|
$j(node).draggable("option","disabled",event.shiftKey || event.metaKey || event.ctrlKey);
|
||||||
$j(this).attr("draggable", event.ctrlKey || event.metaKey ? "true" : "");
|
$j(this).attr("draggable", event.shiftKey || event.metaKey ? "true" : "");
|
||||||
|
|
||||||
// Disabling draggable adds some UI classes, but we don't care so remove them
|
// Disabling draggable adds some UI classes, but we don't care so remove them
|
||||||
$j(node).removeClass("ui-draggable-disabled ui-state-disabled");
|
$j(node).removeClass("ui-draggable-disabled ui-state-disabled");
|
||||||
if(!(event.ctrlKey || event.metaKey) || !this.addEventListener) return;
|
if(!(event.shiftKey || event.metaKey) || !this.addEventListener) return;
|
||||||
})
|
})
|
||||||
.on("dragstart", function(event) {
|
.on("dragstart", function(event) {
|
||||||
if(event.dataTransfer == null) {
|
if(event.dataTransfer == null) {
|
||||||
@ -261,6 +264,15 @@ function egwDragActionImplementation()
|
|||||||
}
|
}
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
//Disable dragging to be able to select content of item by holding Ctrl then selecting content
|
||||||
|
$j(node).off("mousedown")
|
||||||
|
.on("mousedown", function(event) {
|
||||||
|
$j(node).draggable("option","disabled",event.ctrlKey || event.metaKey);
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
$j(node).draggable(
|
$j(node).draggable(
|
||||||
{
|
{
|
||||||
"distance": 20,
|
"distance": 20,
|
||||||
|
Loading…
Reference in New Issue
Block a user