Skip to content

Conversation

@KomatiBhavaniSankar
Copy link

PR Description

This PR fixes the broken carousel on the NeuroML documentation landing page, where only a single image was displayed even though multiple carousel items and navigation controls were defined.

Root Cause of the Issue

The carousel was not functioning correctly due to Bootstrap version mismatches and configuration issues, including:

  • Use of Bootstrap 4 attributes (data-ride, data-target, data-slide, data-slide-to) in a documentation setup that relies on Bootstrap 5
  • A mismatch between the number of carousel indicators (3) and the number of carousel items (5), causing incorrect navigation behavior
  • As a result, the carousel initialized incorrectly and did not cycle through images

Fix Implemented

The following corrections were made to ensure compatibility with Bootstrap 5 and restore proper carousel functionality:

  • Replaced deprecated Bootstrap 4 attributes with Bootstrap 5 equivalents:

    • data-ridedata-bs-ride
    • data-targetdata-bs-target
    • data-slidedata-bs-slide
    • data-slide-todata-bs-slide-to
  • Updated the carousel indicators to correctly match all five carousel items

  • Verified that previous/next navigation controls function correctly

  • Confirmed that all images render and cycle as expected on the landing page

These changes restore the intended carousel behavior on the NeuroML documentation home page. Please let me know if any changes or suggestions are needed; I will be happy to follow them. Thank you.

Fixes #216

Output

Screen.Recording.2026-01-05.012027.mp4

@sanjayankur31
Copy link
Member

Thanks for this @KomatiBhavaniSankar . Could you please clarify if/how AI was used to generate the PR? We're happy for people to use AI in their work and contributions, but we require people to note how AI was used.

@sanjayankur31 sanjayankur31 self-assigned this Jan 5, 2026
@sanjayankur31 sanjayankur31 added the C: content Improvements or additions to documentation label Jan 5, 2026
@github-project-automation github-project-automation bot moved this to 🆕 New in NeuroML backlog Jan 5, 2026
@sanjayankur31 sanjayankur31 moved this from 🆕 New to 👀 In review in NeuroML backlog Jan 5, 2026
@KomatiBhavaniSankar
Copy link
Author

Thanks for this @KomatiBhavaniSankar . Could you please clarify if/how AI was used to generate the PR? We're happy for people to use AI in their work and contributions, but we require people to note how AI was used.

For the debugging and correcting of the code, I will didn't used AI as I already faced similar error in my pervious open source contribution and I figured it out but I used AI for PR description and issue description to avoid my grammatical errors. I might use the AI in future contributions for understanding the codebase and tech stack involved in it and to find efficient way to learn the tech stack incase if i didn't know that tech stack. I hope this will help you guys. I'm also reading the docs to contribute to the NeuroML
and i found this tutorial : https://youtu.be/bGeilKVTajk?si=j9FyWpETltc5Fb2g . If possible please share any resources that might help me to understand the NeuroML. Thank you maintaining the amazing project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C: content Improvements or additions to documentation

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

Carousel broken on docs home page?

2 participants