Skip to content

Conversation

@carmoran0
Copy link
Contributor

Thank you for your contribution! Before sending this Pull Request, please confirm the following:

  • I have read the contributing guidelines
  • I have run python check.py and received a ✅
  • The icon's size is under 1,024 Bytes
  • The layout of the SVG looks like this including newlines
<svg xmlns="http://www.w3.org/2000/svg"
aria-label="..." role="img"
viewBox="0 0 512 512"><path
d="m0 0H512V512H0"
fill="#fff"/> ... </svg>

If you have done the above, please send the Pull Request.

@edent
Copy link
Owner

edent commented Nov 2, 2025

This looks good - can you remove the Last.fm; just one logo per PR please.

@jmb
Copy link
Collaborator

jmb commented Nov 2, 2025

Can you centre it up as well?
image

@carmoran0
Copy link
Contributor Author

image Sure, centered. Sorry about the pr, i thought i created another pr, i dont know how to separate them now :(

@jmb
Copy link
Collaborator

jmb commented Nov 3, 2025

Looks good.

If you either update your branch/fork from the this repo or revert or untrack your copies of the lastfm files, I think that should do it.

@jmb
Copy link
Collaborator

jmb commented Nov 3, 2025

I think you can optimise a bit more too... 😉

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.

3 participants