| Summary: | Double-click does not expand option tree | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Buovjaga <ilmari.lauhakangas> |
| Component: | UI | Assignee: | Not Assigned <libreoffice-bugs> |
| Status: | VERIFIED FIXED | ||
| Severity: | minor | CC: | raykowj, thomas.lendo, xiscofauli |
| Priority: | medium | Keywords: | bibisected, bisected, needUITest, regression |
| Version: | 6.1.0.0.alpha0+ | ||
| Hardware: | x86-64 (AMD64) | ||
| OS: | Linux (All) | ||
| See Also: | https://bugs.documentfoundation.org/show_bug.cgi?id=127935 | ||
| Whiteboard: | target:6.1.0 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 102187 | ||
|
Description
Buovjaga
2018-03-10 13:35:04 UTC
confirm with Version: 6.1.0.0.alpha0+ Build ID: 7c693fc3f7218e1ca0c85a0de76ae84226391256 CPU threads: 8; OS: Linux 4.14; UI render: default; VCL: kde4; Locale: nl-BE (en_US.UTF-8); Calc: group Version: 6.1.0.0.alpha0+ Build ID: 598cf9f76176c6bef00402755d0e941a036c6a2d CPU threads: 8; OS: Linux 4.14; UI render: default; VCL: gtk3; Locale: nl-BE (en_US.UTF-8); Calc: group but not with Version: 6.0.3.0.0+ Build ID: 97b65102db4b1e8f95faf08521329da37cc3f2ec CPU threads: 8; OS: Linux 4.14; UI render: default; VCL: kde4; Locale: nl-BE (en_US.UTF-8); Calc: group Regression introduced by: author Jim Raykowski <raykowj@gmail.com> 2018-01-01 14:52:41 -0900 committer Thorsten Behrens <Thorsten.Behrens@CIB.de> 2018-01-11 11:30:08 +0100 commit 582b2ed5ba25657afc2c2d45860899325b3b2450 (patch) tree 60ec98ab10f80fb2da3a08ab8beb3d73e3c344b7 parent f66fbd947f70f6be6b22ab372facaeb9e2fb63ae (diff) tdf#36308 make double click not expand/collapse node in Navigator tree Bisected with: bibisect-linux64-6.1 Adding Cc: to Jim Raykowski (In reply to Xisco Faulí from comment #2) > Regression introduced by: > > author Jim Raykowski <raykowj@gmail.com> 2018-01-01 14:52:41 -0900 > committer Thorsten Behrens <Thorsten.Behrens@CIB.de> 2018-01-11 11:30:08 > +0100 > commit 582b2ed5ba25657afc2c2d45860899325b3b2450 (patch) > tree 60ec98ab10f80fb2da3a08ab8beb3d73e3c344b7 > parent f66fbd947f70f6be6b22ab372facaeb9e2fb63ae (diff) > tdf#36308 make double click not expand/collapse node in Navigator tree > > Bisected with: bibisect-linux64-6.1 > > Adding Cc: to Jim Raykowski Hi Xisco, Yes this patch did cause a regression but was fixed by the patch for bug 115105. What I can see is happening to cause this now is that double clicks are being seen as single clicks here [1] causing the code that expands/collapses on double click to be skipped over. [1] https://opengrok.libreoffice.org/xref/core/svtools/source/contnr/svimpbox.cxx#2020 *** Bug 116356 has been marked as a duplicate of this bug. *** (In reply to Jim Raykowski from comment #3) > > What I can see is happening to cause this now is that double clicks are > being seen as single clicks here [1] causing the code that expands/collapses > on double click to be skipped over. > > [1] > https://opengrok.libreoffice.org/xref/core/svtools/source/contnr/svimpbox. > cxx#2020 So this isn't due to the double clicks being seen as single clicks but a few lines of code down the DoubleClickHdl() has been recently changed to return false. commit 7651e57573952758032ceb88f16e2dbbb6cc4e18 Author: Jan Holesovsky <kendy@collabora.com>, Thu Mar 1 12:13:53 2018 +0100 (12 days ago) Committer: Jan Holesovsky <kendy@collabora.com>, Thu Mar 1 15:41:13 2018 +0100 (12 days ago) Follows: libreoffice-6-0-branch-point Branches: <Expand> tdf#115950: Indicate that the dialog was already destroyed. The DoubleClickHdl()'s return value does not mean "was handled", but actually "is there anything more to be done"; so if we actually destroy the dialog in the aDoubleClickHdl.Call(this), we have to return 'false'. Change-Id: I7c510e8341eb5f74703b4266f86cb1e840a2b1fd Reviewed-on: https://gerrit.libreoffice.org/50261 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jan Holesovsky <kendy@collabora.com> Thanks for spotting this, pushed a fix to gerrit :-) https://gerrit.libreoffice.org/51211 Jan Holesovsky committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=1b9af08481b8f7f4bd15a30508606dff56b8e74f tdf#116334: Actually when there is no handler, we have to return 'true'. It will be available in 6.1.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Thanks all for pointing out the commit! :-) Fixed now. Verified in Version: 6.1.0.0.alpha0+ Build ID: 8329f4541e27402d19729ae1588af8bfe61f7b49 CPU threads: 4; OS: Linux 4.13; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); Calc: group Thank you Kendy! |