| Summary: | Impress crash on animation search | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | BDF <bugs_documentfoundation_org.5.kuru> |
| Component: | Impress | Assignee: | Caolán McNamara <caolan.mcnamara> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | caolan.mcnamara, raal, telesto |
| Priority: | medium | Keywords: | bibisected, bisected, regression |
| Version: | 7.1.0.3 release | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:7.4.0 target:7.3.4 | ||
| Crash report or crash signature: | Regression By: | ||
| Attachments: | crash test file | ||
|
Description
BDF
2022-05-08 12:09:18 UTC
Created attachment 180000 [details]
crash test file
A simple file to test the crash
Steps:
1) Open file
2) Select square
3) Add object animation
4) Click in list of selectable animations
5) Type to search for something that doesn't exist
6) Crash
2nd test Libre Office: 7.3.2.2 OS: Windows 10 I tested this bug on my Windows machine and it does not crash over there. Unlike Linux, Windows does only search for a single letter you just pressed. So if you type "bug" it jumps to the first result for 'b', then jumps to the first result of 'u' and then the first result of 'g'. If there is no entry starting with said number, it does not do anything. On Linux the letters are kept. So it searches for 'b' then 'bu' and then 'bug'. If there is no entry for the searched term Impress crashes. This also applies if the first letter does not exist (eg. if you type "y" in Impress with german UI). Confirm Version: 7.4.0.0.alpha0+ / LibreOffice Community Build ID: 08c979e235aa993f50e54224f8c715bb70697c33 CPU threads: 4; OS: Linux 5.13; UI render: default; VCL: gtk3 Locale: cs-CZ (cs_CZ.UTF-8); UI: en-US Calc: threaded Jumbo Version: 7.0.0.0.alpha1+ Build ID: 574c57090642347980d2395e1e183cc7b5c171ad CPU threads: 4; OS: Linux 5.13; UI render: default; VCL: gtk3 Locale: cs-CZ (cs_CZ.UTF-8); UI: en-US Calc: threaded In Version: 5.4.0.0.alpha1+ the animation is not searchable, so it should be bibisectable. This seems to have begun at the below commit. Adding Cc: to Caolán McNamara; Could you possibly take a look at this one? Thanks 9ff12c78a14275c103335bce76a0ddee5680ef49 is the first bad commit commit 9ff12c78a14275c103335bce76a0ddee5680ef49 Author: Jenkins Build User <tdf@pollux.tdf> Date: Mon Apr 6 21:44:38 2020 +0200 source 21e8ce9ab9a522d25e95773af59b7ccccaee4e54 https://git.libreoffice.org/core/+/21e8ce9ab9a522d25e95773af59b7ccccaee4e54 weld custom animation panel Caolán McNamara committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/247f4aefb9879dd465e34a2072218a0f1748de53 Resolves: tdf#148985 crash on searching for non-existing prefix It will be available in 7.4.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. done in trunk, backport to 7-3 in gerrit Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-7-3": https://git.libreoffice.org/core/commit/502a45e112cdc5e296311ef8f3c220b706340e06 Resolves: tdf#148985 crash on searching for non-existing prefix It will be available in 7.3.4. 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. I tested this with LibreOfficeDev-7.4.0.0.alpha0_2022-01-23.standard-x86_64.AppImage that I downloaded from https://libreoffice.soluzioniopen.com/daily-version/ Pro: The crash can no longer be reproduced Contra: The search is now just like in Windows. So you have to know where the effect is you want to have or go on a happy scroll tour with your mouse (should this be added as a feature request?) ---------- Version: 7.4.0.0.alpha0+ / LibreOffice Community Build ID: 465c3ad95059f0efa13c8027f7383c4d20a5b2ff CPU threads: 16; OS: Linux 5.13; UI render: default; VCL: kf5 (cairo+xcb) Locale: de-AT (de_AT.UTF-8); UI: de-DE Calc: threaded I moved the suggestion to here: Bug 149015 - Impress: Improve search for animations see https://bugs.documentfoundation.org/show_bug.cgi?id=149015 |