hedgedoc / react-client

The frontend of the upcoming version 2.0
https://hedgedoc.dev
GNU Affero General Public License v3.0
104 stars 13 forks source link

Import brand icons for Element and Discourse #154

Closed mrdrogdrog closed 1 year ago

mrdrogdrog commented 4 years ago

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.

SISheogorath commented 4 years ago

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.

mrdrogdrog commented 4 years ago

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 commented 4 years ago

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

SISheogorath commented 4 years ago

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.

mrdrogdrog commented 4 years ago

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

mrdrogdrog commented 4 years ago

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.

DerMolly commented 4 years ago

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

DerMolly commented 1 year ago

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