Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Use precision select cursor in Screenshots instead of regular arrow #4187

Closed
sevaan opened this issue Mar 5, 2018 · 3 comments
Closed

Use precision select cursor in Screenshots instead of regular arrow #4187

sevaan opened this issue Mar 5, 2018 · 3 comments

Comments

@sevaan
Copy link
Collaborator

sevaan commented Mar 5, 2018

I've seen a few issues about using specific cursors for various edit functions (e.g. a crop cursor when cropping) but can't seem to find an issue about what overall cursor to use, so apologies if this is a repeat.

screen shot 2018-03-05 at 1 46 17 pm

Right now Screenshots uses a regular arrow cursor...I think we should switch this to a precision select cursor as the state of the page has changed and the cursor should reflect that the user is no longer in a normal browsing mode. It also allows users to make more precise selection boxes if drawing their own.

@ghost ghost added this to the Sprint 10 (61-2) ⛅ milestone Mar 15, 2018
@ghost ghost modified the milestones: Sprint 10 (61-2) ⛅, Sprint 11 (61-3) 👗 Apr 16, 2018
@punamdahiya punamdahiya self-assigned this Apr 17, 2018
@punamdahiya
Copy link
Contributor

@sevaan Quick clarification on the requirement here, should the crosshair cursor replaces arrow, when the user clicks edit pencil icon or it's should change only when the user enters edit screen and moves cursor over canvas with screenshot image. Thanks

@punamdahiya punamdahiya added the P1 label Apr 17, 2018
@sevaan
Copy link
Collaborator Author

sevaan commented Apr 18, 2018

This is not a markup mode feature but rather for Screenshots as a whole. This crosshair cursor should be used as soon as the user clicks "Take a screenshot" and is selecting what shot they are going to take.

@sevaan
Copy link
Collaborator Author

sevaan commented Apr 18, 2018

If a user goes hovers over over a button (e.g. Save), we use the hand cursor.

punamdahiya added a commit to punamdahiya/screenshots that referenced this issue Apr 24, 2018
testeaxeax pushed a commit to testeaxeax/screenshots that referenced this issue Jun 7, 2018
testeaxeax pushed a commit to testeaxeax/screenshots that referenced this issue Jun 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants