This is an automated email from the ASF dual-hosted git repository.

gengliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 4748d858b44 [SPARK-44820][DOCS] Switch languages consistently across 
docs for all code snippets
4748d858b44 is described below

commit 4748d858b4478ea7503b792050d4735eae83b3cd
Author: panbingkun <pbk1...@gmail.com>
AuthorDate: Thu Aug 24 17:48:23 2023 -0700

    [SPARK-44820][DOCS] Switch languages consistently across docs for all code 
snippets
    
    ### What changes were proposed in this pull request?
    The pr aims to fix bug for `Switch languages consistently across docs for 
all code snippets`.
    
    ### Why are the changes needed?
    When a user chooses a different language for a code snippet, all code 
snippets on that page should switch to the chosen language. This was the 
behavior for, for example, Spark 2.0 doc: 
https://spark.apache.org/docs/2.0.0/structured-streaming-programming-guide.html
    But it was broken for later docs, for example the Spark 3.4.1 doc: 
https://spark.apache.org/docs/latest/quick-start.html
    We should fix this behavior change and possibly add test cases to prevent 
future regressions.
    
    ### Does this PR introduce _any_ user-facing change?
    No.
    
    ### How was this patch tested?
    Manually test:
    ```
    cd docs
    SKIP_API=1 bundle exec jekyll serve --watch
    ```
    
    ### Was this patch authored or co-authored using generative AI tooling?
    No.
    
    Closes #42657 from panbingkun/SPARK-44820.
    
    Authored-by: panbingkun <pbk1...@gmail.com>
    Signed-off-by: Gengliang Wang <gengli...@apache.org>
---
 docs/js/main.js | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/docs/js/main.js b/docs/js/main.js
index 8426ddc7818..97434146556 100755
--- a/docs/js/main.js
+++ b/docs/js/main.js
@@ -65,12 +65,21 @@ function codeTabs() {
 
       tabBar.append(
         '<li class="nav-item"><button class="' +
-        active + 'nav-link tab_' + lang + '" data-bs-target="#' + id +
-        '" data-bs-toggle="tab">' + buttonLabel + '</button></li>'
+        active + 'nav-link tab_' + lang + '" data-bs-target="#' +
+        id + '" data-tab-lang="tab_' + lang + '" data-bs-toggle="tab">' +
+        buttonLabel + '</button></li>'
       );
     });
     counter++;
   });
+  $("ul.nav-tabs button").click(function (e) {
+    // Toggling a tab should switch all tabs corresponding to the same language
+    // while retaining the scroll position
+    e.preventDefault();
+    var scrollOffset = $(this).offset().top - $(document).scrollTop();
+    $("." + $(this).attr('data-tab-lang')).tab('show');
+    $(document).scrollTop($(this).offset().top - scrollOffset);
+  });
 }
 
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to