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

feat: Add CITATION.cff icon and update fileIcons #2338

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Anselmoo
Copy link
Contributor

@Anselmoo Anselmoo commented May 22, 2024

Description

Add CITATION.cff icon for cff-files for GitHub repos, see also: https://citation-file-format.github.io

Contribution Guidelines

PKief and others added 2 commits May 20, 2024 12:29
* feat: ⚙️ Refactor pattern code structure

* Rename type

* feat: update barrels

* feat: update recommended extensions for development

* Add comment
Copy link

Preview

Thank you for creating a pull request. This preview shows you how your changes will look on the different themes:

Generated Preview

You can find more information how to contribute in the contribution guidelines.

Copy link

Preview

Thank you for creating a pull request. This preview shows you how your changes will look on the different themes:

Generated Preview

You can find more information how to contribute in the contribution guidelines.

Copy link

Preview

Thank you for creating a pull request. This preview shows you how your changes will look on the different themes:

Generated Preview

You can find more information how to contribute in the contribution guidelines.

Also fixing the centering of the icon
Copy link

Preview

Thank you for creating a pull request. This preview shows you how your changes will look on the different themes:

Generated Preview

You can find more information how to contribute in the contribution guidelines.

Copy link

Preview

Thank you for creating a pull request. This preview shows you how your changes will look on the different themes:

Generated Preview

You can find more information how to contribute in the contribution guidelines.

Copy link

Preview

Thank you for creating a pull request. This preview shows you how your changes will look on the different themes:

Generated Preview

You can find more information how to contribute in the contribution guidelines.

@Anselmoo
Copy link
Contributor Author

Preview

Thank you for creating a pull request. This preview shows you how your changes will look on the different themes:

Generated Preview

You can find more information how to contribute in the contribution guidelines.

image

@PKief PKief force-pushed the main branch 2 times, most recently from e748540 to b209d61 Compare May 23, 2024 11:57
Copy link

Preview

Thank you for creating a pull request. This preview shows you how your changes will look on the different themes:

Generated Preview

You can find more information how to contribute in the contribution guidelines.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for the PR. This icon is not fitting into the theme, because it's hardly to read the letters "CFF" and typically I try to avoid using letters at all for the icons. In addition there's no space around it as mentioned in the contributing guidelines. So I doubt it that this icon can be merged to the main branch.

As we already have an icon for this file type, I think we should keep that:
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@PKief, thx for the quick feedback ...

One short question would be a material format-quote " a possibility?

It is just about the visibility of this unique type of file.

@PKief PKief force-pushed the main branch 2 times, most recently from eeb4f0e to b2e0fe4 Compare June 12, 2024 20:23
@PKief PKief force-pushed the main branch 2 times, most recently from 2ff604a to b1ad1d2 Compare July 5, 2024 17:31
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.

None yet

2 participants