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.
   ![Uploading Readme.png…]()
   
   
   - 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. https://github.com/apache/airflow
   2. https://github.com/facebook/flipper
   3. 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

Reply via email to