Skip to content

fix: restrict owl background to index.html, isolate all non-home page classes #181

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
DavidKRK merged 3 commits into from
Apr 16, 2026
Merged

fix: restrict owl background to index.html, isolate all non-home page classes #181

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a01a0e6
fix: restore background-image on index.html, isolate music-page class
Copilot Apr 16, 2026
6796d3e
Update assets/css/style.css
DavidKRK Apr 16, 2026
04804a0
fix: update all non-home pages to use page-specific body classes
Copilot Apr 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions assets/css/style.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -259,6 +259,8 @@ body.home-page {
}
body:not(.home-page) {
background-color: #416e73;
display: flex;
flex-direction: column;
}


Expand Down
2 changes: 1 addition & 1 deletion bio.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
<!-- CSS -->
<link rel="stylesheet" href="assets/css/style.css">
</head>
<body class="home-page">
<body class="bio-page">

<div class="background-image"></div>

Expand Down
2 changes: 1 addition & 1 deletion contact.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@

</head>

<body class="home-page">
<body class="contact-page">

<div class="background-image"></div>

Expand Down
2 changes: 1 addition & 1 deletion event.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
<link rel="stylesheet" href="assets/css/style.css">
<script src="https://cdn.jsdelivr.net/npm/howler@2.2.3/dist/howler.core.min.js"></script>
</head>
<body class="home-page">
<body class="event-page">

<div class="background-image"></div>

Expand Down
2 changes: 1 addition & 1 deletion music.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@
}
</style>
</head>
<body class="home-page">
<body class="music-page">

Copilot AI Apr 16, 2026

Copy link

Choose a reason for hiding this comment

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

The PR description says the owl background is intended only for the home page, but several other pages in the repo still use class="home-page" on <body> (e.g., bio.html, contact.html, event.html, shop.html). If the intent is to restrict the owl background to index.html only, those pages would also need their <body> class updated (or the CSS selector changed to target index.html more specifically).

Copilot uses AI. Check for mistakes.

@DavidKRK DavidKRK Apr 16, 2026

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@claude[agent] @codex[agent] @snyk-bot @copilot apply changes based on this feedback

Copilot AI Apr 16, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Applied in 04804a0. Updated bio.html, contact.html, event.html, and shop.html to use page-specific classes (bio-page, contact-page, event-page, shop-page).

index.html is the only page that keeps class="home-page" and the owl background. All other pages now fall under body:not(.home-page) which applies the teal background (#416e73) and the flex column layout.


<div class="background-image"></div>

Expand Down
2 changes: 1 addition & 1 deletion shop.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
<link rel="stylesheet" href="assets/css/style.css">
<script src="https://cdn.jsdelivr.net/npm/howler@2.2.3/dist/howler.core.min.js"></script>
</head>
<body class="home-page">
<body class="shop-page">

<div class="background-image"></div>

Expand Down
Loading