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

Icon request: file-check, file-x, folder-check, folder-x #529

Open
controversial opened this issue Feb 5, 2019 · 16 comments · May be fixed by #669
Open

Icon request: file-check, file-x, folder-check, folder-x #529

controversial opened this issue Feb 5, 2019 · 16 comments · May be fixed by #669

Comments

@controversial
Copy link
Contributor

controversial commented Feb 5, 2019

Icon Request

  • Icon name: “check” and “x” variants of the file and folder icons
  • Use case: There are already file-plus and file-minus icons (and the equivalent for folders) to symbolize adding and removing these objects, but being able to symbolize success and validation in relation to files and folders would be enormously useful for many apps, especially those involving uploads
  • Screenshots of similar icons:

screen shot 2019-02-04 at 8 43 37 pm

screen shot 2019-02-04 at 8 43 26 pm

(all from unicons)
@jletey
Copy link

jletey commented Jun 23, 2019

@colebemis @locness3 @controversial How does this look?

@moeenio
Copy link

moeenio commented Jun 23, 2019

@colebemis @locness3 @controversial How does this look?

This one looks great, but make sure to be in compliance with #171 .

@jletey
Copy link

jletey commented Jun 23, 2019

This one looks great, but make sure to be in compliance with #171 .

Yup ... looking like it is in compliance ... going to submit a PR!

@moeenio
Copy link

moeenio commented Jun 23, 2019

@johnlestey
No they are not.

@jletey
Copy link

jletey commented Jun 23, 2019

No they are not.

@locness3 What is wrong?

@moeenio
Copy link

moeenio commented Jun 23, 2019

Strokes = Borders. This means you should use borders instead of fills. Also, you shouldn't use the masks provided for Figma when submitting your icons. You should use only svg shapes themselves.

@jletey
Copy link

jletey commented Jun 23, 2019

Refactoring now ... will update you to have a look when I'm done!

@jletey
Copy link

jletey commented Jun 23, 2019

@locness3 I've updated the icons ... please take a look! They now use SVG shapes instead of components and also I have shrunk the "x" to have the same top and bottom space as the "check"

@moeenio
Copy link

moeenio commented Jun 23, 2019

@locness3 I've updated the icons ... please take a look! They now use SVG shapes instead of components and also I have shrunk the "x" to have the same top and bottom space as the "check"

Great, but please use a border for the checks and X, instead of a fill. BTW, you should remove the original "file" and "folder" groups from the icons.

@jletey
Copy link

jletey commented Jun 23, 2019

@locness3 The checks and Xs now have a 1px border (is that a violation of #171?) and I have removed all icon groups ... solely vectors now!

@moeenio
Copy link

moeenio commented Jun 24, 2019

a 1px border (is that a violation of #171?)

#171 is again very clear : "Every line and shape has a 2px center-aligned stroke with round joins and round caps"

@jletey
Copy link

jletey commented Jun 24, 2019

@locness3 Have changed ... but IMHO, they look ugly now

@jletey
Copy link

jletey commented Jul 2, 2019

@locness3 @controversial How does it look now?

Screen Shot 2019-07-02 at 7 31 27 AM

@moeenio
Copy link

moeenio commented Jul 2, 2019

Maybe make the checks and crosses smaller.
Screenshot_2019-07-02 Figma(1)

@jletey
Copy link

jletey commented Jul 8, 2019

@locness3 @ahtohbi4 How does it look now? Figma

Screen Shot 2019-07-08 at 8 00 15 AM

@moeenio
Copy link

moeenio commented Jul 8, 2019

Perfect!

jletey added a commit to jletey/feather that referenced this issue Jul 8, 2019
@jletey jletey linked a pull request Jul 8, 2019 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants