Skip to content

Conversation

@Pilot-Pirx
Copy link
Member

This is a proof of concept how to add a new icon set.

I am not sure if we should go with "Remix" but that needs to be discussed

This is a proof of concept how to add a new icon set.
@Pilot-Pirx Pilot-Pirx marked this pull request as draft May 24, 2024 15:16
@Pilot-Pirx
Copy link
Member Author

Pilot-Pirx commented May 24, 2024

The complete Remix iconset can be found here:
https://remixicon.com/

The icon set is 26x26 for large icons and 16x16 for small icons. Fallback icons are from Industrial icon set.

@Pilot-Pirx Pilot-Pirx requested a review from pedlino May 24, 2024 15:32
@pedlino pedlino marked this pull request as ready for review May 24, 2024 16:21
Copy link
Contributor

@pedlino pedlino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! ;)

@Pilot-Pirx
Copy link
Member Author

Hi Pedro,

technically it is good.
But there are still a lot of icons missing.

I will do a build for Windows soon.

@Pilot-Pirx
Copy link
Member Author

@Pilot-Pirx Pilot-Pirx marked this pull request as draft May 30, 2024 18:53
@Pilot-Pirx
Copy link
Member Author

OK, this PR will never get merged until there is a discussion on the icon theme...

@pedlino
Copy link
Contributor

pedlino commented Sep 20, 2025

OK, this PR will never get merged until there is a discussion on the icon theme...

Discussion where? On the Dev mailing list?

@Pilot-Pirx
Copy link
Member Author

I really don't care *where" it is discussed...
I am only offering the code, if there is no interest I can close this PR.

@pedlino
Copy link
Contributor

pedlino commented Sep 20, 2025

I really don't care *where" it is discussed... I am only offering the code, if there is no interest I can close this PR.

I am interested. If that is not enough, then you can close the PR.

@Pilot-Pirx
Copy link
Member Author

So you are not interested in a discussion to convince others?

OK

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.

2 participants