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

Updating KI theme to KI's recently updated visual identity #361

Merged
merged 8 commits into from
Oct 29, 2022

Conversation

ellessenne
Copy link
Contributor

@ellessenne ellessenne commented Oct 20, 2022

  • Sign the CLA (contributor license agreement). The CLA assistant will remind you below if you have not signed the agreement before.
  • Add a news item to NEWS.md and your name to DESCRIPTION (by alphabetical order) unless you have already been listed there.
  • Provide a URL to a live example (and even better, a couple of screenshots) if you are contributing a new theme.

Hi,
With this PR I am updating the KI theme to the recently updated visual identity.
It is still work in progress so please do not merge yet – I am creating this to keep track of things and to upload a new, updated logo created specifically for digital channels.
I will give a couple of presentations next week using this template, so I'll confirm it's okay to merge after that if that's okay 😄

Thanks!

Alessandro

@ellessenne
Copy link
Contributor Author

New logo:

KI_digital_logotyp_negativ_RGB

@CLAassistant
Copy link

CLAassistant commented Oct 20, 2022

CLA assistant check
All committers have signed the CLA.

Copy link
Owner

@yihui yihui left a comment

Choose a reason for hiding this comment

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

Okay, just let me know when it's ready to merge. Thanks!

@ellessenne
Copy link
Contributor Author

Hi @yihui,
I tested the updated theme this week and would be happy for this to be merged now.
Thanks!

Alessandro

Copy link
Owner

@yihui yihui left a comment

Choose a reason for hiding this comment

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

Perfect. Thanks!

@yihui yihui merged commit 7b69a3d into yihui:master Oct 29, 2022
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.

3 participants