react-client icon indicating copy to clipboard operation
react-client copied to clipboard

Import brand icons for Element and Discourse

Open mrdrogdrog opened this issue 4 years ago • 7 comments

Which part of the project should be enhanced? The landing page footer.

Is your enhancement request related to a problem? Please describe. Fork awesome doesn't provide icons for element or discourse.

Describe the solution you'd like We could import the icons as svgs into our project and add them to the footer.

Describe alternatives you've considered Using the fork awesome replacement icons.

mrdrogdrog avatar Jun 05 '20 14:06 mrdrogdrog

I recommend to open an issue in the fork-awesome project: https://github.com/ForkAwesome/Fork-Awesome/issues

There is a matrix icon, I just think it sometimes looks imperfect because the square bracket are a bit thin.

SISheogorath avatar Jun 05 '20 14:06 SISheogorath

We could replace the riot link with https://matrix.to/#/#codimd:matrix.org, because that's the way to go now. Then we could replace the icon with the matrix icon, that is already included.

mrdrogdrog avatar Jun 05 '20 14:06 mrdrogdrog

For discourse: https://github.com/ForkAwesome/Fork-Awesome/issues/243

mrdrogdrog avatar Jun 05 '20 14:06 mrdrogdrog

The reason for the riot instead of matrix.to link down there is mainly to make it a bit easier for people who are not into the ecosystem yet. I never considered matrix.to a rather useful webpage for people who aren't already aware of how matrix works.

Even when we use the matrix icon, I think we could use the riot link there.

SISheogorath avatar Jun 05 '20 14:06 SISheogorath

A PR for the Riot icon was already opened. https://github.com/ForkAwesome/Fork-Awesome/pull/232

mrdrogdrog avatar Jun 05 '20 14:06 mrdrogdrog

The last release of fork awesome was in Feb 2019! I propose to implement our own solution with the svgs until fork awesome has included these icons.

mrdrogdrog avatar Jun 05 '20 14:06 mrdrogdrog

I'd suggest leaving this issue open until we can use the new fork awesome icons

DerMolly avatar Jun 05 '20 14:06 DerMolly

This issue is irrelevant because we will use another icon font see https://github.com/hedgedoc/hedgedoc/issues/2929

DerMolly avatar Nov 22 '22 20:11 DerMolly