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] Own icon for fonts folders #585

Closed
2 tasks done
totkeks opened this issue Dec 29, 2016 · 6 comments · Fixed by #623
Closed
2 tasks done

[Icon Request] Own icon for fonts folders #585

totkeks opened this issue Dec 29, 2016 · 6 comments · Fixed by #623
Milestone

Comments

@totkeks
Copy link

totkeks commented Dec 29, 2016

  • I'm sure this issue is not a duplicate?
  • I want to create an icon request:
    - Type: folder
    - Icon Name: FontFolder
    - Sample original Icon: Icon (hint: does not display in browser but pops up a download window)
    - Extensions: fonts
@jens1o
Copy link
Member

jens1o commented Dec 29, 2016

Great idea, but I think this icon does not match our style guidelines. Your thoughts @robertohuertasm and @JimiC ?

@JimiC
Copy link
Member

JimiC commented Dec 29, 2016

See this comment.

@totkeks
Copy link
Author

totkeks commented Dec 29, 2016

@jens1o Was just trying to give an example. Basically the folder icons you got now and then this "Aa" kind of symbol in front of them.

@JimiC
Copy link
Member

JimiC commented Jan 11, 2017

I think that changing the folder icons will not have too much value as we also use those icons as a background for the specific named folder icons.
Having a watermark on the folder icon and on top the folder specific icon will definitely bloat the icon.

@totkeks
Copy link
Author

totkeks commented Jan 11, 2017

@JimiC I think you might misunderstand me. Of course I want to keep things as they are, with the folder background and then the specific named folder icons on top of them. Just one more for fonts.
image
Like in this picture. Though instead of the little image icon in front of the folder icon, just put a little "Aa" or something that represents fonts in general very well.

@JimiC
Copy link
Member

JimiC commented Jan 11, 2017

Oh for a folder containing fonts? I totally misread you if this is the case. If so it's totally doable.

@JimiC JimiC added this to the 6.1.0 milestone Jan 11, 2017
@JimiC JimiC mentioned this issue Jan 12, 2017
5 tasks
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.

3 participants