Copilot commented on code in PR #441:
URL: https://github.com/apache/cloudstack-www/pull/441#discussion_r3504128429
##########
src/components/Events/index.tsx:
##########
@@ -62,6 +50,18 @@ involved with the community.
<div className="horizontal-card">
<div className="hc-header">Past Events:</div>
+ <div className="row">
+ <div className="col-lg-9 pt-3 px-4">
+ <img src="/img/CSEUG-2026-05-21.png" alt=""
className="events-img events-overview-img"/>
+ <p className="title mt-2">CloudStack European User Group
2026</p>
+ <p className="events-info"><img src="img/pin_icon.png"
className="me-2" alt=""/>the Hague, the Netherlands</p>
Review Comment:
Location name should be capitalized as “The Hague” (proper noun); this also
matches how it’s written in the corresponding roundup blog post.
##########
src/components/Events/index.tsx:
##########
@@ -62,6 +50,18 @@ involved with the community.
<div className="horizontal-card">
<div className="hc-header">Past Events:</div>
+ <div className="row">
+ <div className="col-lg-9 pt-3 px-4">
+ <img src="/img/CSEUG-2026-05-21.png" alt=""
className="events-img events-overview-img"/>
+ <p className="title mt-2">CloudStack European User Group
2026</p>
+ <p className="events-info"><img src="img/pin_icon.png"
className="me-2" alt=""/>the Hague, the Netherlands</p>
+ <p className="events-info"><img src="img/calendar_icon.png"
className="me-2" alt=""/>May 21, 2026</p>
+ </div>
+ <div className="col-lg-3 px-4 events-btn-margin d-flex
justify-content-center justify-content-lg-end align-items-center">
+ <a href="/blog/cseug26-roundup" target="_blank"
target="_blank" className="btn btn-outline-secondary me-3" >Read More</a>
+ <a
href="https://www.youtube.com/watch?v=GTN-l-B-iPY&list=PLnIKk7GjgFlbrStHohANRr8-IuX4EE1pq";
target="_blank" className="btn btn-primary me-3">Watch</a>
Review Comment:
This block has a duplicate target attribute (invalid HTML) and the
target="_blank" links are missing rel="noopener noreferrer". Remove the
duplicate attribute and add rel to avoid reverse-tabnabbing.
##########
src/components/Events/index.tsx:
##########
@@ -42,7 +30,7 @@ involved with the community.
<p className="events-info"><img src="img/calendar_icon.png"
className="me-2" alt=""/>October 11-14, 2026</p>
</div>
<div className="col-lg-3 px-4 events-btn-margin d-flex
justify-content-center justify-content-lg-end align-items-center">
- <a href="
https://communityovercode.org/call-for-presentations/"; target="_blank"
className="btn btn-primary me-3" >Submit</a>
+ <a
href="https://web.cvent.com/event/ac71ce47-2b5f-424c-abfe-5b48255315fb/register";
target="_blank" className="btn btn-primary me-3" >Register</a>
Review Comment:
External links opened with target="_blank" should include rel="noopener
noreferrer" to prevent reverse-tabnabbing and to match existing link hygiene
used elsewhere in the site.
##########
blog/2026-05-29-cseug26-roundup/index.md:
##########
@@ -17,12 +17,10 @@ the continued growth and momentum of the CloudStack
community.
<!-- truncate -->
-<!--
<div class="col col-3 col-lg text-center">
- <a class="button button--primary" href="" target="_blank">Watch Session
Recordings</a>
+ <a class="button button--primary"
href="https://www.youtube.com/watch?v=GTN-l-B-iPY&list=PLnIKk7GjgFlbrStHohANRr8-IuX4EE1pq";
target="_blank">Watch Session Recordings</a>
Review Comment:
Links opened with target="_blank" should include rel="noopener noreferrer"
to prevent reverse-tabnabbing (this pattern is already used in other blog posts
in this repo).
--
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]
