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

hoverCursor overrided by "move" cursor on click and drag #2360

Closed
nodegin opened this issue Jul 25, 2015 · 6 comments
Closed

hoverCursor overrided by "move" cursor on click and drag #2360

nodegin opened this issue Jul 25, 2015 · 6 comments

Comments

@nodegin
Copy link

nodegin commented Jul 25, 2015

c = new fabric.Circle { radius: 16, fill: 'rgba(255, 255, 255, .4)', left: cx - 8, top: cy - 8, opacity: .5 }
c.hasControls = c.hasBorders = no
c.lockMovementX = c.lockMovementY = yes
c.hoverCursor = 'default'
@canvas.add c

It will add a circle with "default" cursor, but if I click on it and try to drag

the cursor will become "move"? How can I fix that

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@asturur
Copy link
Member

asturur commented Apr 25, 2016

That is default behavour, drag cursor on fabricJs is not customizable.

@nodegin
Copy link
Author

nodegin commented Apr 26, 2016

Is it due to browser limitation or something else? It just looks odd since
I have locked the X and Y movement, it would be nice if there's a new feature
to override the drag cursor, thanks.

@asturur
Copy link
Member

asturur commented Apr 26, 2016

i marked as possible feature indeed.
I'm releaseing a little bit of fixes here and there.
I will release 1.6.1 and then i will work on control customization, in that part there may be also cursor customization.

@nodegin
Copy link
Author

nodegin commented Apr 26, 2016

Sounds good👍

@asturur
Copy link
Member

asturur commented Apr 30, 2016

I understad that bad.
Because i did not read the code ( specify pleaseeee )
You meant that if the object is LOCKED the cursor should not change to moveCursor, because is not gonna move.
Is customizable in that way.

@nodegin
Copy link
Author

nodegin commented May 1, 2016 via email

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

No branches or pull requests

3 participants