refactor: optimize site#237
Conversation
Aarebecca
commented
Apr 16, 2026
Aarebecca
commented
- Optimize image language
- Optimize stream demo style
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the home page and documentation by externalizing image URLs in translations, adopting the AVIF format for architecture diagrams, and refining the layout and scroll behavior of the streaming syntax showcase. Feedback suggests removing redundant scrollbar-hiding styles from the <pre> element, as the parent container already manages this behavior.
| <pre | ||
| className="text-[13px] leading-relaxed text-secondary dark:text-secondary-dark font-mono p-4 md:p-6 m-0 whitespace-pre scrollbar-hide" | ||
| style={{ | ||
| scrollbarWidth: 'none', | ||
| msOverflowStyle: 'none', | ||
| }}> | ||
| {displayCode} | ||
| </pre> |
There was a problem hiding this comment.
The scrollbar-hide class and the associated inline styles on the <pre> element are redundant. The scroll container is the parent <div> (line 634), which already handles scrollbar hiding. Since the <pre> element does not have its own overflow property set to auto or scroll, it will not produce scrollbars, making these styles unnecessary on this specific element.
<pre className="text-[13px] leading-relaxed text-secondary dark:text-secondary-dark font-mono p-4 md:p-6 m-0 whitespace-pre">
{displayCode}
</pre>
1 participant
