Browse Source

Fixed onScrollBody event and added parameter

zhixin 6 years ago
parent
commit
e0f6e1efc1
2 changed files with 3 additions and 3 deletions
  1. 1 1
      site/docs/api/events.md
  2. 2 2
      src/bootstrap-table.js

+ 1 - 1
site/docs/api/events.md

@@ -369,7 +369,7 @@ $('#table').on('event-name.bs.table', function (e, arg1, arg2, ...) {
 
 
 - **jQuery Event:** `scroll-body.bs.table`
 - **jQuery Event:** `scroll-body.bs.table`
 
 
-- **Parameter:**: `undefined`
+- **Parameter:**: `$tableBody` (from 1.15.5)
 
 
 - **Detail:**
 - **Detail:**
 
 

+ 2 - 2
src/bootstrap-table.js

@@ -1909,8 +1909,6 @@ class BootstrapTable {
   horizontalScroll () {
   horizontalScroll () {
     // horizontal scroll event
     // horizontal scroll event
     // TODO: it's probably better improving the layout than binding to scroll event
     // TODO: it's probably better improving the layout than binding to scroll event
-
-    this.trigger('scroll-body')
     this.$tableBody.off('scroll').on('scroll', ({currentTarget}) => {
     this.$tableBody.off('scroll').on('scroll', ({currentTarget}) => {
       if (this.options.showHeader && this.options.height) {
       if (this.options.showHeader && this.options.height) {
         this.$tableHeader.scrollLeft($(currentTarget).scrollLeft())
         this.$tableHeader.scrollLeft($(currentTarget).scrollLeft())
@@ -1919,6 +1917,8 @@ class BootstrapTable {
       if (this.options.showFooter && !this.options.cardView) {
       if (this.options.showFooter && !this.options.cardView) {
         this.$tableFooter.scrollLeft($(currentTarget).scrollLeft())
         this.$tableFooter.scrollLeft($(currentTarget).scrollLeft())
       }
       }
+
+      this.trigger('scroll-body', $(currentTarget))
     })
     })
   }
   }