palexdev / VirtualizedFX

GNU Lesser General Public License v3.0
46 stars 4 forks source link

Exception on scrolling since 11.9.1 #12

Closed Alexander-Ploskin closed 1 year ago

Alexander-Ploskin commented 1 year ago

MFXScrollBar throws an exception when VirtualScrollPane.isAutoHideBars is true since 11.9.1. I think it is related to MFXEffects, but I have no idea how to fix it.

palexdev commented 1 year ago

What exception can you post the stacktrace? I cannot test it right now but I may tell you why looking at the exception

Alexander-Ploskin commented 1 year ago

java.lang.IllegalArgumentException: Cannot register this When construct as the given observable is already being observed io.github.palexdev.mfxcore.observables.When.register(When.java:108) io.github.palexdev.mfxcore.observables.OnChanged.listen(OnChanged.java:192) io.github.palexdev.mfxcore.observables.OnChanged.listen(OnChanged.java:59) io.github.palexdev.virtualizedfx.controls.skins.VirtualScrollPaneSkin.lambda$addListeners$19(VirtualScrollPaneSkin.java:355) com.sun.javafx.binding.ExpressionHelper$SingleChange.fireValueChangedEvent(ExpressionHelper.java:181) com.sun.javafx.binding.ExpressionHelper.fireValueChangedEvent(ExpressionHelper.java:80) javafx.beans.property.ReadOnlyBooleanPropertyBase.fireValueChangedEvent(ReadOnlyBooleanPropertyBase.java:78) javafx.beans.property.ReadOnlyBooleanWrapper.fireValueChangedEvent(ReadOnlyBooleanWrapper.java:103) javafx.beans.property.BooleanPropertyBase.markInvalid(BooleanPropertyBase.java:111) javafx.beans.property.BooleanPropertyBase.set(BooleanPropertyBase.java:145) javafx.scene.Node.setHover(Node.java:7244) javafx.scene.Node.preprocessMouseEvent(Node.java:8495) javafx.scene.Node.lambda$buildEventDispatchChain$3(Node.java:8857) com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114) com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56) com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114) com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56) com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114) com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56) com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114) com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56) com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114) com.sun.javafx.event.EventUtil.fireEventImpl(EventUtil.java:74) com.sun.javafx.event.EventUtil.fireEvent(EventUtil.java:54) javafx.event.Event.fireEvent(Event.java:198) com.sun.javafx.event.EventQueue.fire(EventQueue.java:48) javafx.scene.Scene$MouseHandler.handleEnterExit(Scene.java:3808) javafx.scene.Scene$MouseHandler.process(Scene.java:3879) javafx.scene.Scene.processMouseEvent(Scene.java:1887) javafx.scene.Scene$ScenePeerListener.mouseEvent(Scene.java:2620) com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:411) com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:301) java.base/java.security.AccessController.doPrivileged(AccessController.java:399) com.sun.javafx.tk.quantum.GlassViewEventHandler.lambda$handleMouseEvent$2(GlassViewEventHandler.java:450) com.sun.javafx.tk.quantum.QuantumToolkit.runWithoutRenderLock(QuantumToolkit.java:424) com.sun.javafx.tk.quantum.GlassViewEventHandler.handleMouseEvent(GlassViewEventHandler.java:449) com.sun.glass.ui.View.handleMouseEvent(View.java:551) com.sun.glass.ui.View.notifyMouse(View.java:937) com.sun.glass.ui.win.WinApplication._runLoop(Native Method) com.sun.glass.ui.win.WinApplication.lambda$runLoop$3(WinApplication.java:184)

palexdev commented 1 year ago

@Alexander-Ploskin Yes this is a known issue: Discussion The fix is ready on the MFXCore side, need to push and then update VirtualizedFX

Alexander-Ploskin commented 1 year ago

Thank you, waiting for the fix.

palexdev commented 1 year ago

Fixed in latest release