-
Notifications
You must be signed in to change notification settings - Fork 565
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: Cursors over UI elements should indicate clickable or text entry #1573
Conversation
I'm not sure what to expect. Are there any before/after screenshots? I don't particularly know what "the proper type" means or how I would know that a cursor is the wrong type. Is there a way we could provide any more description in here? |
@dmsnell There were essentially no pointer cursors over any clickable elements. I'll 👀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is already a huge improvement! 👏 I noticed a few places that still need attention:
- The note list on the sidebar (to switch notes, or to pin/unpin a note -- should be pointer, is default)
- Tags at the bottom of a note, when it has tags (should be pointer, is text cursor)
- "Add a tag..." at the bottom of a note (should be text cursor, is default -- only in Firefox, it's correct in Safari)
This should have resolved that https://github.com/Automattic/simplenote-electron/pull/1573/files#diff-b4aef3f86a137f3edaf5a4e2fbb2a23fR104 Do you not see the pointer
Made Tags that are on a note pointer since clicking on them removes from note. In the UI spring cleaning, this interaction will be cleaned up. Where you enter new tags I changed the cursor to text. |
…lectron into fix/cursor-types
Okay so for whatever reason it's not working to add it to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great now!
Fix
Cursors in the UI do not use the proper type to indicate when elements are clickable or editable. This PR fixes those issue across the app.
Test
Review
Only one developer is required to review these changes, but anyone can perform the review.
Release
RELEASE-NOTES.txt
was updated with: