swikarpat commented on code in PR #18066: URL: https://github.com/apache/kafka/pull/18066#discussion_r1876354093
########## README.md: ########## @@ -1,5 +1,14 @@ -Apache Kafka -================= +# Apache Kafka + +<a href="https://kafka.apache.org/"> +<picture> + <source media="(prefers-color-scheme: dark)" srcset="docs/images/kafka-logo-readme-dark.svg"> + <source media="(prefers-color-scheme: light)" srcset="docs/images/kafka-logo-readme-light.svg"> + <img alt="Kafka Logo" src="docs/images/kafka-logo-readme-light.svg" width="80"> +</picture> +</a> +<br><br> Review Comment: @cadonna I sincerely apologize if my previous comment came across as upset or unprofessional. That was certainly not my intention, and I value our collaboration on this PR. I understand and respect the Apache Kafka project's commitment to finding consensus and agreement before merging changes. I appreciate you taking the time to provide your feedback and explain your perspective. I'm happy to continue the discussion and address any concerns you have. Attached is screenshot currently it looks like. <img width="900" alt="Readme" src="https://github.com/user-attachments/assets/a7365ba2-ddfe-4e5a-9f56-98afa243ed20"> - I've removed "About Apache Kafka" TITLE and also removed "Prerequisite" TITLE - I am providing three examples, each have different logo positions. No.1 & 3 below has position similar to I proposed. examples- 1. [Airflow](https://github.com/apache/airflow) 2. [Flipper](https://github.com/facebook/flipper) 3. [Kubeflow](https://github.com/kubeflow/kubeflow) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org