Latest commit 4.0.2 overrides cursor

dimmank commented 7 years ago

I'm submitting a ... [ x] bug report [ ] feature request [ ] question about the decisions made in the repository
Do you want to request a feature or report a bug?

Bug

What is the current behavior?

The cursor style (pointer) is forced to pointer on elements using the dnd-draggable, dnd-droppable directives. <element [style.cursor]='pointer'>

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar.

Update from version 2.2.2 to 4.0.2

What is the expected behavior?

Expected behavior should be adding a class that sets the cursor, giving the developer the option to override with a simple css change. Adding a DragDropConfig should not be required.

What is the motivation / use case for changing the behavior?

Never to force a style on to a component

Please tell us about your environment:

Windows 10

Angular version: 4.0.1
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

JavanXD commented 7 years ago

I made it work by adding a !important to my child-element but this is bad style. grafik

.myDrager {
    font: normal normal normal 14px/1 FontAwesome;
    color: #868e96;
    cursor: move !important;
}
.myDrager::after {
    content: '\f0c9';
}

Avd6977 commented 7 years ago

Any traction on this bug? Still present in 4.2.0

daveo1001 commented 6 years ago

Also interested.

stewhouston commented 6 years ago

+1. This probably shouldn't be the default behavior.

I was able to override it by explicitly setting [style.cursor]="'default'" on the dnd-droppable.

akserg / ng2-dnd

Latest commit 4.0.2 overrides cursor #135