Re: [SITE-IMPROVEMENT] Issue need to confirm about the API web

2022-01-08 Thread Stamatis Zampetakis
The easiest would be to wrap code with {% highlight java %} / {% highlight bash %}, which is also used in various other places of the .md file already, instead of back-ticks. Like that there will be a horizontal scroll bar when needed. Alternatively as Julian mentioned we can change the wrapping b

Re: [SITE-IMPROVEMENT] Issue need to confirm about the API web

2022-01-07 Thread Julian Hyde
You’re right, those 3 examples don’t look great. They all seem to be caused by Jekyll’s code block (back-ticks), e.g. https://github.com/apache/calcite/blame/cbda134d8e75a8a5c63a1aa9bcddab6fc6756021/site/_docs/howto.md#L73

Re:[SITE-IMPROVEMENT] Issue need to confirm about the API web

2022-01-07 Thread Zhe Hu
Hi, I have one small suggestion for the SITE_IMPROVEMENT. Recently,when I browse howto.html through Chrome Explorer, I’ve found that some lines are too long, which causes a little bit format disorder(just based on my personal preference). I’ll pick several cases in howto.md below: (1) Line 73: Ho

Re: [SITE-IMPROVEMENT] Issue need to confirm about the API web

2021-12-30 Thread Julian Hyde
+1 > On Dec 28, 2021, at 7:11 PM, Zhe Hu wrote: > > +1 for removing the redundant lowercase “calcite”/”calcite-avatica”. > > > Best, > ZheHu > > > > > On 12/29/2021 10:44,Chunwei Lei wrote: > Sounds reasonable to me. > > > Best, > Chunwei > > > On Wed, Dec 29, 2021 at 6:44 AM xiong dua

Re: [SITE-IMPROVEMENT] Issue need to confirm about the API web

2021-12-28 Thread Zhe Hu
+1 for removing the redundant lowercase “calcite”/”calcite-avatica”. Best, ZheHu On 12/29/2021 10:44,Chunwei Lei wrote: Sounds reasonable to me. Best, Chunwei On Wed, Dec 29, 2021 at 6:44 AM xiong duan wrote: After checking the regenerate the Web, I noticed a small problem with the Calc

Re: [SITE-IMPROVEMENT] Issue need to confirm about the API web

2021-12-28 Thread Chunwei Lei
Sounds reasonable to me. Best, Chunwei On Wed, Dec 29, 2021 at 6:44 AM xiong duan wrote: > After checking the regenerate the Web, I noticed a small problem with the > Calcite API URI https://calcite.apache.org/javadocAggregate/ . The doc > title should be "Apache Calcite API", but is "Apache