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

feat: Add mouse-text icon #897

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

csandman
Copy link
Contributor

Closes #508

This PR is for my version of a mouse cursor when hovering over text.

Here is a scaled up view of it:
image

And here is a real size version:

mouse-text

I believe it follows all of the guidelines from this post - #171 (comment)

If there is anything that should be changed, please let me know! I will change anything as needed asap.

@codecov
Copy link

codecov bot commented Apr 28, 2020

Codecov Report

Merging #897 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #897   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         5           
  Lines           34        34           
  Branches         3         3           
=========================================
  Hits            34        34           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b15b4a7...52d1791. Read the comment docs.

@csandman csandman changed the title Add mouse-text icon feat: Add mouse-text icon Apr 28, 2020
@csandman csandman changed the title feat: Add mouse-text icon feat: Add mouse-text icon Apr 28, 2020
Copy link

@locness3 locness3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasn't this already submited ? Anyway, looks good. But maybe not calling it mouse-text, maybe cursor-text

@csandman
Copy link
Contributor Author

@locness3 I had posted in a comment but that thread seemed to die so I figured I would submit a PR. And I'd be cool with that name, I was trying to keep it consistent with the other mouse icon (mouse-pointer). I can change it to whatever works best!

@ericfennis
Copy link

@csandman Nice love this icon.
Do like to contribute this icon to featherity ? Or may we use this one there?

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

Successfully merging this pull request may close these issues.

Icon Request: Cursor
3 participants