Skip to content

Conversation

@MattyTheHacker
Copy link
Member

not done yet

@MattyTheHacker MattyTheHacker self-assigned this Jan 18, 2026
@MattyTheHacker MattyTheHacker requested a review from a team as a code owner January 18, 2026 13:05
Copilot AI review requested due to automatic review settings January 18, 2026 13:05
@github-actions github-actions bot added the sync Keep this pr up to date with it's base branch automatically label Jan 18, 2026
@cssbhamdev cssbhamdev enabled auto-merge (squash) January 18, 2026 13:05
@github-actions
Copy link

This pull request has been marked to automatically sync to its base branch. You can disable this behavior by removing the label.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds support for the CSS Ball 2026 event by creating a new set of pages and styles. The PR includes a main event page, menu page, SCSS stylesheet, and YAML data file with event details.

Changes:

  • Added 2026 ball event pages with venue information, tickets, FAQ, and interactive map
  • Created dedicated SCSS stylesheet with purple/gold color scheme for the 2026 ball theme
  • Added YAML data file containing event details, menu options, ticket prices, and FAQ content

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 10 comments.

File Description
css/ball/2026/main.scss New stylesheet defining the visual theme for the 2026 ball with custom colors, layout styles, and responsive design
ball/2026/menu.html Menu page template displaying the ball's food options with dietary information
ball/2026/index.html Main event page with venue details, ticket information, countdown timer, and interactive map
_data/ball/2026.yaml Event data including dates, pricing, menu items, and frequently asked questions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


&.small {
border: 1px solid rgba(255, 255, 255, 0.3);
position: block;
Copy link

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The position property value 'block' is invalid. The position property accepts values like 'static', 'relative', 'absolute', 'fixed', or 'sticky', but not 'block'. If the intent is to change the display type, use 'display: block' instead, or if reverting position, use 'position: static'.

Suggested change
position: block;
display: block;

Copilot uses AI. Check for mistakes.
<h3 id="count"><br></h3>

{% for link in ball.links %}
<a href='{{link.url}}' class="button {% if link.url == '' %}empty"{% else %}" href='{{link.url}}' {% endif %}>
Copy link

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 'href' attribute is duplicated and incorrectly placed. The first href is already set before the class attribute ends, and then it appears again after the closing quote. This creates invalid HTML syntax with a malformed attribute structure.

Suggested change
<a href='{{link.url}}' class="button {% if link.url == '' %}empty"{% else %}" href='{{link.url}}' {% endif %}>
<a href="{{ link.url }}" class="button{% if link.url == '' %} empty{% endif %}">

Copilot uses AI. Check for mistakes.
</h2>

{% for link in ball.menu-links %}
<a href='{{link.url}}' class="button {% if link.url == '' %}empty"{% else %}" href='{{link.url}}' {% endif %}>
Copy link

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 'href' attribute is duplicated and incorrectly placed. The first href is already set before the class attribute ends, and then it appears again after the closing quote. This creates invalid HTML syntax with a malformed attribute structure.

Suggested change
<a href='{{link.url}}' class="button {% if link.url == '' %}empty"{% else %}" href='{{link.url}}' {% endif %}>
<a class="button{% if link.url == '' %} empty{% endif %}" href='{{link.url}}'>

Copilot uses AI. Check for mistakes.
<hr class="bleed-bottom">
<div class="text-center-div margin-top margin-bottom section-info" id="venue">
<h2 class="subtitle mobile-only">The Venue</h2>
<img class="image" src="/assets/images/ball-25-venue.jpg" alt="Image of the Horton Suite, where the CSS Ball is being held.">
Copy link

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The image source refers to 'ball-25-venue.jpg', which appears to be an image from the 2025 ball. For the 2026 ball, this should either be updated to use a 2026-specific venue image or verified that the same venue image is intentionally being reused.

Suggested change
<img class="image" src="/assets/images/ball-25-venue.jpg" alt="Image of the Horton Suite, where the CSS Ball is being held.">
<img class="image" src="/assets/images/ball-26-venue.jpg" alt="Image of the Horton Suite, where the CSS Ball is being held.">

Copilot uses AI. Check for mistakes.
<hr class="bleed-top">
<div class="section-tickets" id="tickets">
<div class="icon">
<img class="icon" src="/assets/images/ball-2025-ticket-icon.png", alt="A cartoon image of a ticket with the text 'CSS Ball, Fazeley Studios, 26 April, Admit One'"/>
Copy link

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comma should be replaced with the word 'alt' to properly separate the 'src' and 'alt' attributes. HTML attributes should be space-separated, and using a comma here creates a syntax error.

Suggested change
<img class="icon" src="/assets/images/ball-2025-ticket-icon.png", alt="A cartoon image of a ticket with the text 'CSS Ball, Fazeley Studios, 26 April, Admit One'"/>
<img class="icon" src="/assets/images/ball-2025-ticket-icon.png" alt="A cartoon image of a ticket with the text 'CSS Ball, Fazeley Studios, 26 April, Admit One'"/>

Copilot uses AI. Check for mistakes.
MattyTheHacker and others added 2 commits January 18, 2026 13:15
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement sync Keep this pr up to date with it's base branch automatically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants