<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>That's what I guessed before having to run. Please submit upstream</div><div id="AppleMailSignature"><br></div><div id="AppleMailSignature">/D</div><div id="AppleMailSignature"><br></div><div id="AppleMailSignature"><br><br>-- <div>Sent from my phone</div></div><div><br>On Feb 5, 2016, at 14:39, Rick Walsh <<a href="mailto:rickmwalsh@gmail.com">rickmwalsh@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 6 February 2016 at 09:15, Rick Walsh <span dir="ltr"><<a href="mailto:rickmwalsh@gmail.com" target="_blank">rickmwalsh@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On 6 February 2016 at 09:10, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>On Sat, Feb 06, 2016 at 08:57:49AM +1100, Rick Walsh wrote:<br>
> On 6 February 2016 at 08:48, Dirk Hohndel <<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>> wrote:<br>
><br>
> > On Sat, Feb 06, 2016 at 07:40:35AM +1100, Rick Walsh wrote:<br>
> > ><br>
> > > It does have the bug where you need to press edit a dozen times before<br>
> > you<br>
> > > can edit a dive.<br>
> ><br>
> > I hate Heisenbugs. So I modified the mobile components to print out debug<br>
> > information whenever the ActionButton is tapped. And of course with that<br>
> > in place it always works, reliably, on the first attempt :-(<br>
> ><br>
> > I have now added this to the next official build just to see if this is<br>
> > just a fluke here or if this really fixes things for everyone. Same<br>
> > visible version number (as this is a change to the mobile components).<br>
> > Please re-download<br>
> > downloads/daily/Subsurface-mobile-4.5.2.774-Qt5.5.arm.apk<br>
> > and test if the "press edit takes many many attempts" still happens for<br>
> > you. If it does, I'd love to see the logcat output...<br>
> ><br>
><br>
> You're easily pleased. On first try it took 15 attempts to press the button<br>
<br>
</span>Now I'm confused again. What else is new. Next attempt. Can you download<br>
the latest one and try (stupidly still with the same filename - sorry)<br>
<span><font color="#888888"><br></font></span></blockquote><div><br></div></span><div>Worked first time, and several times in a row after that.<br><br>D/Subsurface(16684): qrc:////imports/org/kde/plasma/mobilecomponents/private/ActionButton.qml:87 (onClicked): qml: ActionButton clicked QQuickAction(0xdb34f0a0)function() { [code] }<br>D/Subsurface(16684): qrc:////imports/org/kde/plasma/mobilecomponents/private/ActionButton.qml:99 (onClicked): qml: triggering action<br>W/Subsurface(16684): (null):0 ((null)): Could not resolve property : linearGradient4538<br>W/Subsurface(16684): (null):0 ((null)): Could not resolve property : linearGradient4588<span class=""><font color="#888888"><br> <br></font></span></div><span class=""><font color="#888888"><div>Rick<br></div></font></span></div></div></div></blockquote><div><br><br></div><div>I'm not sure if you've already worked out what I just did, but I added a little debug output myself to find out where the (mouse.x < buttonGraphics.x || mouse.x > buttonGraphics.x + buttonGraphics.width) condition was going wrong.<br><br>D/Subsurface(24634): qrc:////imports/org/kde/plasma/mobilecomponents/private/ActionButton.qml:88 (onClicked): qml: outside zone - mouse.x: 719 ; buttonGraphics.x: 738 ; buttonGraphics.width: 114<br>D/Subsurface(24634): qrc:////imports/org/kde/plasma/mobilecomponents/private/ActionButton.qml:88 (onClicked): qml: outside zone - mouse.x: 680 ; buttonGraphics.x: 738 ; buttonGraphics.width: 114<br>D/Subsurface(24634): qrc:////imports/org/kde/plasma/mobilecomponents/private/ActionButton.qml:88 (onClicked): qml: outside zone - mouse.x: 721 ; buttonGraphics.x: 738 ; buttonGraphics.width: 114<br>D/Subsurface(24634): qrc:////imports/org/kde/plasma/mobilecomponents/private/ActionButton.qml:88 (onClicked): qml: outside zone - mouse.x: 693 ; buttonGraphics.x: 738 ; buttonGraphics.width: 114<br>D/Subsurface(24634): qrc:////imports/org/kde/plasma/mobilecomponents/private/ActionButton.qml:91 (onClicked): qml: within zone - mouse.x: 757 ; buttonGraphics.x: 738 ; buttonGraphics.width: 114<br> <br></div></div>It appears that the button is centre aligned, so buttonGraphics.x relates to the middle of the button. Click just to the left of the centre and it fails, click just to the right and it works.<br><br></div><div class="gmail_extra">I've fixed the problem with this little change:<br><br>$ diff ../subsurface/qt-mobile/qml/mobilecomponents/private/ActionButton.qml components/mobilecomponents/qml/private/ActionButton.qml <br>87c87<br>< if (mouse.x < buttonGraphics.x - buttonGraphics.width / 2 || mouse.x > buttonGraphics.x + buttonGraphics.width / 2) {<br>---<br>> if (mouse.x < buttonGraphics.x || mouse.x > buttonGraphics.x + buttonGraphics.width) {<br><br><br></div><div class="gmail_extra">Did you come to the same conclusion? I'm guessing you've already worked out a solution, but if not and you think that's correct, I'll submit that upstream.<br><br></div><div class="gmail_extra">Rick<br></div><div class="gmail_extra"><br><br></div></div>
</div></blockquote></body></html>