Skip to content

Commit

Permalink
fix: unable to pan when page is invisible
Browse files Browse the repository at this point in the history
  • Loading branch information
bubkoo committed Jul 16, 2020
1 parent 69036e4 commit a7995ae
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 6 deletions.
2 changes: 1 addition & 1 deletion packages/x6/src/addon/scroller/index.less
Expand Up @@ -37,7 +37,7 @@
}
}

&-pannable {
&-panning {
user-select: none;
}
}
Expand Down
20 changes: 15 additions & 5 deletions packages/x6/src/addon/scroller/index.ts
Expand Up @@ -186,10 +186,20 @@ export class Scroller extends View {
}

protected onBackgroundEvent(e: JQuery.TriggeredEvent) {
const isBackgroundEvent = this.options.background
? this.$background.is(e.target)
: this.$content.is(e.target)
if (isBackgroundEvent) {
let shouldHnadleEvent = false

const target = e.target

if (!this.options.pageVisible) {
shouldHnadleEvent =
this.graph.view.background === target || this.graph.view.grid === target
} else if (this.options.background) {
shouldHnadleEvent = this.background === target
} else {
this.content === target
}

if (shouldHnadleEvent) {
const handler = this.delegatedHandlers[e.type]
if (typeof handler === 'function') {
handler.apply(this.graph, arguments)
Expand Down Expand Up @@ -1126,7 +1136,7 @@ export namespace Scroller {

namespace Util {
export const containerClass = 'graph-scroller'
export const panningClass = `${containerClass}-pannable`
export const panningClass = `${containerClass}-panning`
export const pagedClass = `${containerClass}-paged`
export const contentClass = `${containerClass}-content`
export const backgroundClass = `${containerClass}-background`
Expand Down

0 comments on commit a7995ae

Please sign in to comment.