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

Add Codeberg Logo #94

Merged
merged 2 commits into from
Nov 14, 2023
Merged

Add Codeberg Logo #94

merged 2 commits into from
Nov 14, 2023

Conversation

uwearzt
Copy link
Contributor

@uwearzt uwearzt commented Jun 29, 2023

Add Logo for codeberg.org

@Finii
Copy link
Collaborator

Finii commented Nov 3, 2023

The icon is the special purpose icon from the official icon kit.

https://codeberg.org/Codeberg/Design/issues/46

@Finii Finii self-requested a review November 14, 2023 08:50
README.md Outdated
@@ -42,6 +42,7 @@ Available logos are:
| Artix Linux | `fl-artix` | `<i class="fl-artix"></i>` | `0xf31f` | <img src="vectors/artix.svg" width="24">
| Budgie | `fl-budgie` | `<i class="fl-budgie"></i>` | `0xf320` | <img src="vectors/budgie.svg" width="24">
| CentOS | `fl-centos` | `<i class="fl-centos"></i>` | `0xf304` | <img src="vectors/centos.svg" width="24">
| Codeberg | `fl-codeberg` | `<i class="fl-codeberg"></i>` | `0xf330` | <img src="vectors/codeberg.svg" width="24">
Copy link
Collaborator

Choose a reason for hiding this comment

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

If we pull this the README.md change will be immediately visible, although the icon is not yet in the release.
This can lead to confusion.

(The gh-pages are very old.)

[why]
When we add icons one by one we usually already assign a codepoint and
change the readme page, but the actual release did not happen yet. So
documenation and real files get out of sync.

[how]
Mark the added but not released icons in the readme table.

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
@Finii
Copy link
Collaborator

Finii commented Nov 14, 2023

Thanks for the nice PR!

@Finii Finii merged commit d1f1042 into lukas-w:master Nov 14, 2023
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