AemieJ edited a comment on pull request #423: URL: https://github.com/apache/camel-website/pull/423#issuecomment-654193808
I have a few comments on these changes : * You have only included the first child of the toolbar list and neglected the rest. However, if you search for `Bean`, there is Bean under `Components` as well as `EIP`. Hence, I suggested including the links of the toolbar to avoid any cases of such behavior. If we were to use specific selectors, there would be no loophole. If there is another possible workaround and it works, that's good then. * Another suggestion: Initially we made changes to the docs pages so that the title wasn't the only thing shown in the search result. I believe we can do the same with blog pages. The hierarchy I would prefer seeing the Blog as - ``` # This being for the blog pages Blog ------- Category Name > Title of the blog ``` * Another case that is not taken care of is that when you search a particular blog, it gives results from the `https://camel.apache.org/category/<title>` as well. To pass this case, I was thinking that we can't avoid crawling on categories as it could be the page where the user wants to land on however to avoid confusion on whether it is the blog page or the category page, we could provide specific layout such as - ``` # This being for the category pages Category Name ------- Title of the Blog ``` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
