Skip to content

Conversation

@hakman
Copy link
Member

@hakman hakman commented Oct 24, 2020

Ref: #10077

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Oct 24, 2020
@hakman hakman changed the title Simplify the kops logo [WIP] Simplify the kops logo Oct 24, 2020
@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 24, 2020
@hakman hakman marked this pull request as draft October 24, 2020 03:07
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hakman

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added area/documentation approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Oct 24, 2020
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 24, 2020
@hakman hakman force-pushed the update-logo branch 2 times, most recently from 2ad5aab to 005951a Compare October 24, 2020 04:18
@johngmyers
Copy link
Member

I showed the old and suggested logo to someone with graphic design experience and got this feedback:

  • The O does not read as capital.
  • What is the purpose of the chevron? In the current logo it appears to incorporate a K, but that is not necessary in the new logo.
  • What is the purpose of the shading?
  • The kerning needs to be adjusted.

What is the typeface? Are there any particular color or design standards we are trying to follow? What are we trying to achieve (other then de-emphasizing the K)?

@johngmyers
Copy link
Member

Perhaps we could use as a design element a more modern helm, without protruding spokes?

@hakman
Copy link
Member Author

hakman commented Oct 24, 2020

@johngmyers anything that looks better?
kOps-logos

@olemarkus
Copy link
Member

If you want to emphasise the ops bit, maybe stylise it kOPS.

@hakman
Copy link
Member Author

hakman commented Oct 24, 2020

If you want to emphasise the ops bit, maybe stylise it kOPS.

That would look like this:
kOPS

@johngmyers
Copy link
Member

The feedback I got was:

  • The kerning is much improved.
  • The helm/gear graphics still aren't reading as O. There is a strong tendency to scan as one syllable and the busy graphics distract from the intent to break that.
  • The gear element is appropriate for "operations"

I suggested using the gear as an O and got "...maybe."

I suggested using a punctuation graphic between the k and the O and got a better reception. "Perhaps a nonagon dot"

@hakman
Copy link
Member Author

hakman commented Oct 25, 2020

Thanks for the feedback @johngmyers. Will do one more iteration.

@hakman
Copy link
Member Author

hakman commented Oct 27, 2020

kOps-logo

@hakman
Copy link
Member Author

hakman commented Oct 27, 2020

@justinsb will look into this, but will reopen if ever needed.
/close

@k8s-ci-robot
Copy link
Contributor

@hakman: Closed this PR.

In response to this:

@justinsb will look into this, but will reopen if ever needed.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@hakman hakman deleted the update-logo branch October 30, 2020 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/documentation cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants