Summary: | LibreOfficeDev 6.5.0.0 Navigator collapse headings works only for sub-headings | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Christopher J Poor <cjpoor> |
Component: | Writer | Assignee: | Jim Raykowski <raykowj> |
Status: | VERIFIED FIXED | ||
Severity: | normal | CC: | dgp-mail, raykowj, suokunlong, xiscofauli |
Priority: | medium | ||
Version: | 7.0.0.0.alpha0+ | ||
Hardware: | x86-64 (AMD64) | ||
OS: | macOS (All) | ||
Whiteboard: | target:6.5.0 target:7.0.0 | ||
Crash report or crash signature: | Regression By: | ||
Bug Depends on: | |||
Bug Blocks: | 103030 | ||
Attachments: |
Crash report sent to Apple
demonstration of root node collapse |
Description
Christopher J Poor
2020-01-22 13:31:36 UTC
Created attachment 157328 [details]
Crash report sent to Apple
Second crash report sent to Apple and copies of opencl_devices.log and opencl_profile.xml available.
Jim, I think you might be interested in this issue. cc: Hi Christopher, I repro the collapse not happening on the headings root when in headings content navigation view mode but not in full content mode. Looks like this has always been the behaviour. Perhaps for headings content navigation view mode, collapse done on the root should collapse to all first level children of the root? Thoughts? Here is a patch that collapses root node to children when in content navigation view: https://gerrit.libreoffice.org/c/core/+/87316 Hi Jim. I would suggest that using the "collapse all" shortcut the second time should do the exact opposite of the "expand all" (I think you had that in the "tool tip") The "collapse all" should work in this way in whichever context the "expand all" works - I think it makes sense for this to be in the navigation view with headings only displayed - but could be in full content mode also. If you are able to code an intermediate expand - to all first level children - that would be great too. However I think it is important, if (1).you have two shortcuts, that the same shortcut does expand/collapse to the same level or (2). you go the org-mode route so that perhaps you hold down a leader key and repeated taps of the secondary key cycle through the visibility levels. What do you think? I will try the dev package again tomorrow and see if it crashes less. Just saw your patch. I will check it out shortly. thanks. (In reply to Christopher J Poor from comment #6) > Just saw your patch. I will check it out shortly. thanks. The patch hasn't been merged yet. You would need to build from source and then apply the patch see what it does. Need more input before merging. I will make a screen capture of what it does. It won't solve the crash/freeze you have been experiencing, see bug 130155 for a report and fix for that. Oh right. I'm not quite up to speed on how to apply a patch. I should spend more time on such things :-) Created attachment 157381 [details]
demonstration of root node collapse
HTH explain the patch
That looks cool! Jim Raykowski committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/7f59313a1bd88dd142ff0b4c9b8cfc7e4beeabc7 tdf#130122 Make content navigation view collapse uniform It will be available in 6.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. It works nicely now on Mac Version: 7.0.0.0.alpha0+ Build ID: 5d7a526ee784abc3d83835b9e43047baf5dbe0b9 CPU threads: 4; OS: Mac OS X 10.13.6; UI render: default; VCL: osx; Locale: en-NZ (en_NZ.UTF-8); UI-Language: en-US Calc: threaded - but not on Linux Mint - I guess I should file another bug for that. @Christopher, Thanks for confirming the Mac build is working. Perhaps the build you tested on Mint isn't patched? I see there is no appimage of 7.0 and the most recent master build for linux rpm is 2020-1-25 which won't have this patch merge on 2020-1-28. (In reply to Jim Raykowski from comment #13) I confirm this is fixed on master, on Linux as well. The bug reporter reported another bug in bug 130315, but used an older version in the test. I closed that bug as WORKSFORME. Jim Raykowski committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/6cb9b06432434fb3257118743780828b3b57326a tdf#130122 Make Navigator CNV collapse to children It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |