showuon commented on a change in pull request #11794:
URL: https://github.com/apache/kafka/pull/11794#discussion_r811537471



##########
File path: docs/toc.html
##########
@@ -45,6 +45,10 @@
                 <li><a href="#producerconfigs">3.3 Producer Configs</a>
                 <li><a href="#consumerconfigs">3.4 Consumer Configs</a>
                 <li><a href="#connectconfigs">3.5 Kafka Connect Configs</a>
+                    <ul>
+                        <li><a href="#sourceconnectconfigs">Source Connector 
Configs</a>
+                        <li><a href="#sinkconnectconfigs">Sink Connector 
Configs</a>
+                    </ul>

Review comment:
       OK, I checked the other sub category links, like: 
    - https://kafka.apache.org/documentation/#os 
    - https://kafka.apache.org/documentation/#basic_ops_add_topic
   They all jump to the sub category directly, while there are still some 
paragraph above it. So, I agree with you. We can add sub TOC: `Source Connector 
Configs` and `Sink Connector Configs` into `Kafka Connect Configs`, and there 
keep the rest as is. 
   Thanks.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to