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 Kotlin icon #862

Merged
merged 1 commit into from
Sep 7, 2022
Merged

Add Kotlin icon #862

merged 1 commit into from
Sep 7, 2022

Conversation

Finii
Copy link
Collaborator

@Finii Finii commented Jul 8, 2022

Description

Add Kotlin icon to Seti & Custom glyph source font.

Requirements / Checklist

What does this Pull Request (PR) do?

Add a hand drawn Kotlin icon to the original-source font.

How should this be manually tested?

Any background context you can provide?

What are the relevant tickets (if any)?

Feature request #861

Screenshots (if appropriate or helpful)

image

@Finii
Copy link
Collaborator Author

Finii commented Jul 8, 2022

This clashes with #833 - codepoints 'overlap'

And I wonder still ... is there no automaton for original-source.otf? :-/

@danielcherubini
Copy link

Is there anything that needs help on this? It would be great to get this icon in there

@Finii
Copy link
Collaborator Author

Finii commented Aug 29, 2022

Still busy with Release 2.2.x aftermath.
Lets wait a bit before working on new additions.

[why]
It's not available anywhere else (it seems).

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

Finii commented Sep 7, 2022

Rebase on master - needed to shift codepoint of new glyph - force push

@Finii Finii merged commit 5226e04 into master Sep 7, 2022
@Finii Finii deleted the feature/add-kotlin-icon branch September 7, 2022 04:26
@4e554c4c 4e554c4c mentioned this pull request Sep 15, 2022
2 tasks
LNKLEO pushed a commit to LNKLEO/Nerd that referenced this pull request Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants