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

Request: foobar2000 #5356

Closed
mfarragher opened this issue Mar 26, 2021 · 7 comments · Fixed by #10529
Closed

Request: foobar2000 #5356

mfarragher opened this issue Mar 26, 2021 · 7 comments · Fixed by #10529
Labels
new icon Issues or pull requests for adding a new icon

Comments

@mfarragher
Copy link

Brand Name: foobar2000
Website: https://www.foobar2000.org
Alexa rank: 99,128

Official resources for icon and color:

Official forum is on HydrogenAudio

Icons:
2014 Aliencat
https://hydrogenaud.io/index.php?topic=104579.0
https://commons.wikimedia.org/wiki/File:Foobar2000_logo_2014.png

https://hydrogenaud.io/index.php?topic=55604.0

@mfarragher mfarragher added the new icon Issues or pull requests for adding a new icon label Mar 26, 2021
@adamrusted
Copy link
Member

adamrusted commented Mar 27, 2021

The PS file available at your last link looks to be the best (and most up to date) version of the logo. There are some strokes to be careful of on the white sections, but otherwise could be quite an easy one to contribute. I'd say (purely from looking at the site and logo) that we should use #000000 as our hex - but happy for you to suggest otherwise. Would you like to give it a go by following our contribution guidelines, @mfarragher?

@mfarragher
Copy link
Author

I have a bit of spare time the next few days so I will give it a try and see how it goes. 🤞

@mfarragher
Copy link
Author

I'm a novice with vector graphics and did tinkering with Inkscape so I'm pleased with how this is rendering 😄

<svg role="img" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24"><title>foobar2000 icon</title><path d="M12 24c-.007.01-5.785-8.674-6.382-9.57-1.622-2.431-.846-4.664 0-6.81L8.619 0l2.396 5.166c.509-.175 1.461-.174 1.97 0L15.381 0l3.001 7.62c.846 2.146 1.622 4.379 0 6.81L12 24"/><path d="M9.626 11.19c1.248 1.41 1.698 3.064 1 3.681-.696.618-2.283-.028-3.532-1.438-1.248-1.41-1.698-3.063-1-3.68.696-.618 2.283.028 3.532 1.438" fill="#ffffff"/><path d="M9.626 11.19c1.248 1.41 1.698 3.064 1 3.681-.696.618-2.283-.028-3.532-1.438-1.248-1.41-1.698-3.063-1-3.68.696-.618 2.283.028 3.532 1.438z" fill="none" stroke="#ffffff" stroke-width=".003"/><path d="M12.407 16.537l1.547 2.686c.24.416-.4.756-.626.364L12 17.283l-1.33 2.307c-.229.396-.859.028-.63-.368l1.556-2.688a.472.472 0 01.81.003" fill="#ffffff"/><path d="M12.407 16.537l1.547 2.686c.24.416-.4.756-.626.364L12 17.283l-1.33 2.307c-.229.396-.859.028-.63-.368l1.556-2.688a.472.472 0 01.81.003z" fill="none" stroke="#ffffff" stroke-width=".003"/><path d="M14.374 11.19c-1.248 1.41-1.698 3.064-1 3.681.696.618 2.283-.028 3.532-1.438 1.248-1.41 1.698-3.063 1-3.68-.696-.618-2.283.028-3.532 1.438" fill="#ffffff"/><path d="M14.374 11.19c-1.248 1.41-1.698 3.064-1 3.681.696.618 2.283-.028 3.532-1.438 1.248-1.41 1.698-3.063 1-3.68-.696-.618-2.283.028-3.532 1.438z" fill="none" stroke="#ffffff" stroke-width=".003"/></svg>

Before doing a PR is that code OK?
Maybe there's a way to simplify those paths and the tags but that's the limit of what I can do with vector graphics.

@PeterShaggyNoble
Copy link
Member

Nice work, @mfarragher, that looks like a good start to me If you let us know what software you're using we can help you merge the paths into one single path to get it ready for PR.

@PeterShaggyNoble PeterShaggyNoble added the awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed label Apr 20, 2021
@mfarragher
Copy link
Author

Inkscape.

I had tried methods like Union on the paths but not sure how to combine them. 🙂

@PeterShaggyNoble
Copy link
Member

OK, so, using your current version as the starting point:

  1. Select the 3 white paths and unify them (Path -> Union or Ctrl + +)
  2. Select the resulting path and the main path of the head
  3. Get the difference of the 2 paths (Path -> Difference or Ctrl + -)
  4. Select all other paths bar the main one and remove them

And that should do it. Any other issues we can address during the review process.

@mfarragher mfarragher mentioned this issue May 15, 2021
3 tasks
@mfarragher
Copy link
Author

Thanks I've made a PR - I've mentioned this issue in there so it's linked above.

@PeterShaggyNoble PeterShaggyNoble removed the awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed label May 15, 2021
@adamrusted adamrusted linked a pull request May 17, 2021 that will close this issue
3 tasks
@adamrusted adamrusted removed a link to a pull request Aug 17, 2023
3 tasks
@adamrusted adamrusted changed the title foobar2000 audio player Request: foobar2000 Feb 17, 2024
@adamrusted adamrusted mentioned this issue Feb 24, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants