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

Added forward icon #578

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

dustinlocke
Copy link

Added a forward icon to go with reply and reply-all

@codecov
Copy link

codecov bot commented Apr 15, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #578   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  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 8f65819...8b7cda1. Read the comment docs.

@colebemis
Copy link
Member

This looks great! Like I mentioned in your other pull, after you run npm run build I think this is good to go 👍

#574 (comment)

@dustinlocke
Copy link
Author

Done.

@mittalyashu
Copy link
Contributor

Can you increase the height of this title a little more?

image

@mittalyashu
Copy link
Contributor

There is already a similar PR for the icon #574

@moeenio
Copy link

moeenio commented Jul 22, 2019

Yes, and both this one and the one you linked are made by @dustinlocke

@ericfennis
Copy link

@dustinlocke Can we use this icon in @featherity ? Or maybe you want to open a PR your self.

@dustinlocke
Copy link
Author

@dustinlocke Can we use this icon in @featherity ? Or maybe you want to open a PR your self.

@ericfennis what's the difference between Featherity and Feather?

@moeenio
Copy link

moeenio commented Jun 30, 2020

Featherity is a community-run fork :P

@ericfennis
Copy link

Yeah what @locness3 says.
The author of feather don't merge any icons the last years. A shame for all the work people put in making icons and try to contribute.

So thats why featherity is started.
So new icons are welcome!

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.

5 participants