Load release list only when needed

Author: GuillaumeGomez

src/web/crate_details.rs

@@ -373,6 +373,53 @@ pub(crate) async fn crate_details_handler(
     Ok(res.into_response())
 }
 
+#[derive(Debug, Clone, PartialEq, Serialize)]
+struct ReleaseList {
+    releases: Vec<Release>,
+    crate_name: String,
+}
+
+impl_axum_webpage! {
+    ReleaseList = "rustdoc/releases.html",
+    cpu_intensive_rendering = true,
+}
+
+#[tracing::instrument]
+pub(crate) async fn get_all_releases(
+    Path(params): Path<CrateDetailHandlerParams>,
+    Extension(pool): Extension<Pool>,
+) -> AxumResult<AxumResponse> {
+    let releases: Vec<Release> = spawn_blocking({
+        let pool = pool.clone();
+        let params = params.clone();
+        move || {
+            let mut conn = pool.get()?;
+            let query = "
+            SELECT
+                crates.id AS crate_id
+            FROM crates
+            WHERE crates.name = $1;";
+
+            let rows = conn.query(query, &[&params.name])?;
+
+            let result = if rows.is_empty() {
+                return Ok(Vec::new());
+            } else {
+                &rows[0]
+            };
+            // get releases, sorted by semver
+            releases_for_crate(&mut *conn, result.get("crate_id"))
+        }
+    })
+    .await?;
+
+    let res = ReleaseList {
+        releases,
+        crate_name: params.name,
+    };
+    Ok(res.into_response())
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;

src/web/routes.rs

@@ -167,6 +167,10 @@ pub(super) fn build_axum_routes() -> AxumRouter {
             "/crate/:name",
             get_internal(super::crate_details::crate_details_handler),
         )
+        .route(
+            "/:name/releases",
+            get_internal(super::crate_details::get_all_releases),
+        )
         .route_with_tsr(
             "/crate/:name/:version",
             get_internal(super::crate_details::crate_details_handler),

static/menu.js

@@ -5,6 +5,34 @@ const updateMenuPositionForSubMenu = (currentMenuSupplier) => {
     subMenu?.style.setProperty('--menu-x', `${currentMenu.getBoundingClientRect().x}px`);
 }
 
+function generateReleaseList(data, crateName) {
+}
+
+let loadReleases = function() {
+    const releaseListElem = document.getElementById('releases-list');
+    // To prevent reloading the list unnecessarily.
+    loadReleases = function() {};
+    if (!releaseListElem) {
+        // We're not in a documentation page, so no need to do anything.
+        return;
+    }
+    const crateName = window.location.pathname.split('/')[1];
+    const xhttp = new XMLHttpRequest();
+    xhttp.onreadystatechange = function() {
+        if (xhttp.readyState !== XMLHttpRequest.DONE) {
+          return;
+        }
+        if (xhttp.status === 200) {
+            releaseListElem.innerHTML = xhttp.responseText;
+        } else {
+            console.error(`Failed to load release list: [${xhttp.status}] ${xhttp.responseText}`);
+            document.getElementById('releases-list').innerHTML = "Failed to load release list";
+        }
+    };
+    xhttp.open("GET", `/${crateName}/releases`, true);
+    xhttp.send();
+};
+
 // Allow menus to be open and used by keyboard.
 (function() {
     var currentMenu;
@@ -53,6 +81,7 @@ const updateMenuPositionForSubMenu = (currentMenuSupplier) => {
         currentMenu = newMenu;
         newMenu.className += " pure-menu-active";
         backdrop.style.display = "block";
+        loadReleases();
     }
     function menuOnClick(e) {
         if (this.getAttribute("href") != "#") {

templates/rustdoc/releases.html

@@ -0,0 +1,6 @@
+{% import "macros.html" as macros %}
+{% set target = "" %}
+{% set inner_path = releases[0].target_name ~ "/index.html" %}
+<ul class="pure-menu-list">
+{{ macros::releases_list(name=crate_name, releases=releases, target=target, inner_path=inner_path) }}
+</ul>

templates/rustdoc/topbar.html

@@ -134,11 +134,8 @@
                             <li class="pure-menu-heading">Versions</li>
 
                             <li class="pure-menu-item">
-                                <div class="pure-menu pure-menu-scrollable sub-menu" tabindex="-1">
-                                    <ul class="pure-menu-list">
-                                        {# Display all releases of this crate #}
-                                        {{ macros::releases_list(name=krate.name, releases=krate.releases, target=target, inner_path=inner_path) }}
-                                    </ul>
+                                <div class="pure-menu pure-menu-scrollable sub-menu" id="releases-list" tabindex="-1">
+                                    <span class="rotate">{{ "spinner" | fas }}</span>
                                 </div>
                             </li>
                         </ul>

templates/style/_navbar.scss

@@ -15,6 +15,15 @@ body {
     padding: 0;
 }
 
+@keyframes rotating_text {
+    from {
+        transform: rotate(0deg);
+    }
+    to {
+        transform: rotate(360deg);
+    }
+}
+
 div.nav-container {
     // Nothing is supposed to be over or hovering the top navbar. Maybe add a few others '('? :)
     z-index: 999;
@@ -324,6 +333,14 @@ div.nav-container {
             }
         }
     }
+
+    #releases-list {
+        .rotate {
+            display: inline-block;
+            font-size: 30px;
+            animation: rotating_text 2s linear infinite;
+        }
+    }
 }
 
 #nav-search {