Copilot commented on code in PR #156:
URL: https://github.com/apache/pinot-site/pull/156#discussion_r2772282150
##########
app/layout.tsx:
##########
@@ -122,6 +122,11 @@ _paq.push(['enableLinkTracking']);
</SearchProvider>
</div>
</ThemeProviders>
+ {/* Scarf Pixel */}
+ <img
+ referrerPolicy="no-referrer-when-downgrade"
+
src="https://static.scarf.sh/a.png?x-pxid=3dd7caad-22f5-40d7-bc2e-20c8eaf91659";
Review Comment:
The Scarf tracking pixel is missing an alt attribute. While tracking pixels
are typically invisible, adding alt="" explicitly indicates this is a
decorative image and improves accessibility compliance.
```suggestion
src="https://static.scarf.sh/a.png?x-pxid=3dd7caad-22f5-40d7-bc2e-20c8eaf91659";
alt=""
```
##########
components/AnnouncementBar.tsx:
##########
@@ -53,7 +66,7 @@ export default function AnnouncementBar({
{text}
</span>
</div>
- {buttonText}
+ <span className="ml-2">{buttonText}</span>
Review Comment:
The buttonText is now wrapped in a span with ml-2 margin, but this creates
visual spacing even when buttonText is empty or undefined. Consider
conditionally rendering this span only when buttonText has a value.
```suggestion
{buttonText && <span
className="ml-2">{buttonText}</span>}
```
##########
data/authors/qiaochu.mdx:
##########
@@ -0,0 +1,12 @@
+---
+name: Qiaochu Liu
+avatar: /static/images/logomark.svg
+occupation: Staff Software Engineer
+company: Uber
+email: [email protected]
+twitter: https://twitter.com/Twitter
+linkedin: https://www.linkedin.com
+github: https://github.com
Review Comment:
The author profile contains placeholder values for email and social media
links. These should be replaced with actual contact information or removed if
unavailable.
```suggestion
```
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
![https://static.scarf.sh/a.png?x-pxid=3dd7caad-22f5-40d7-bc2e-20c8eaf91659"](https://static.scarf.sh/a.png?x-pxid=3dd7caad-22f5-40d7-bc2e-20c8eaf91659"){kind=link}