Skip to content

Commit 7167aee

Browse files
committed
fix: remove manual Bootstrap modal initialization
- Fix aria-hidden accessibility warning by letting Bootstrap manage modals automatically - Modal functionality preserved through data-attributes (data-bs-toggle, data-bs-target) - Resolves focus management conflicts between manual JS and Bootstrap's built-in handlers
1 parent a9f92a5 commit 7167aee

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

src/js/main.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,8 @@ dayjs.extend(utc);
88

99
// Initialize when DOM is loaded
1010
document.addEventListener('DOMContentLoaded', () => {
11-
// Initialize all Bootstrap components that need JavaScript
12-
const modals = document.querySelectorAll('.modal');
13-
modals.forEach(modalEl => new bootstrap.Modal(modalEl));
11+
// Bootstrap modals are initialized automatically via data-attributes
12+
// No manual initialization needed for aria-hidden compatibility
1413
const leoBirthday = new Date(document.getElementById('leo-birthday').dataset.leoBirthday);
1514
const leoDaysUntilNextBirthday = daysUntilNextBirthday(leoBirthday);
1615
const today = new Date();

0 commit comments

Comments
 (0)