Skip to content
This repository has been archived by the owner on Aug 31, 2021. It is now read-only.

Bugfix 22798 #2136

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Bugfix 22798 #2136

wants to merge 4 commits into from

Conversation

macMikey
Copy link

mobile datagrid scrollers are initialized with
delayTouches = true
this will prevent a datagrid from scrolling because the moment the datagrid is touched, a row will be selected.

livecodeali and others added 4 commits May 13, 2020 13:51
This patch ensures that the key field of the custom properties
editor in the property inspector is only selected in response to
a change in the tree view hilite.
[[ Bug 22285 ]] Restrict selection of key field in customprops editor
library sets delayTouches to false which will mean that scrolling a dg will be impossible, because the moment the dg is touched a row will be selected.
@@ -368,7 +368,7 @@ private command _CreateMobileScroller
mobileControlSet sScrollerId, "canBounce", "true"
mobileControlSet sScrollerId, "pagingEnabled", "false"
mobileControlSet sScrollerId, "canScrollToTop", "true"
mobileControlSet sScrollerId, "delayTouches", "false"
mobileControlSet sScrollerId, "delayTouches", "true"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this rather be merged into develop-9.6 to up its priority/urgency?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, I didn't see a 9.6 branch open and
nobody else commented on the how list and
nobody else commented on the bug report and
this has been like this since whenever dg2 came out but nobody else seemed to notice, so I'm going with "maybe".
I mean, I personally think it should be a hotfix and get a special code name and a bug bounty Bond car to me and a lifetime free subscription to every LC premium thing, but I'm also megalomaniac.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with Mikey: this bug is driving me nuts.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't hold your breath waiting for a pull request to be reviewed.
Meanwhile, this is a stack-only script, so you can always patch your own copy.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants