Bug 358449 - 15.13 git master: transition list not properly reset when clearing filter input field while effects(!) list has filter "show video only" active
Summary: 15.13 git master: transition list not properly reset when clearing filter inp...
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (other bugs)
Version First Reported In: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-23 20:40 UTC by Wegwerf
Modified: 2016-02-05 15:06 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Wegwerf 2016-01-23 20:40:49 UTC
When entering something into the filter input field in the "Transitions" pane, the list of transitions gets filtered according to the input. Now, if you clear the input field or edit the input, the list does not get updated properly. It seems to get updated only when adding input, not when editing or removing input.

Reproducible: Always

Steps to Reproduce:
1. Start entering some transition prefix into the input field of the Transitions pane. E.g., "alpha"
2. Delete some trailing chars or delete all of the input field text.


Actual Results:  
List of transitions does not update.

Expected Results:  
List of transitions should update.
Comment 1 Unknown 2016-01-24 03:05:03 UTC
@wegwerf, I just tested it on Kdenlive-git master 15.13.0.r6951.6704725-1 on Antergos Linux, and it works fine. Typing in "alpha", the pressing backspace on the characters updates the list. Also, clicking the "clear" button on the far right of the field updates it fine as well.
Comment 2 Wegwerf 2016-01-24 08:37:44 UTC
@Jesse, thank you for trying to reproduce. Now this is strange.
Comment 3 Wegwerf 2016-01-24 21:33:31 UTC
Still persisting with recent git pull; make clean; cmake; make cycle. Totally strange.
Comment 4 Unknown 2016-01-26 20:22:33 UTC
@wegerf, you bet. :) Curious, which distro are you using?
Comment 5 Wegwerf 2016-01-26 20:32:27 UTC
It's Kubuntu 15.10 with all recent updates installed, but not Kdenlive ;) Qt is reported as 5.4.2, against which I also built Kdenlive. KF is 5.15.0.
Comment 6 Wegwerf 2016-01-26 20:41:12 UTC
Whoa! I've finally found out how to reproduce this and on what particular state it depends. And this is totally bizarre, believe me...

0. So, please make the Effects and Transitions panes both visible at the same time for better, well, effect (pun intented).
1. Next, in the Effects(!) pane click on "Show video effects" (NOT: "Show all effects").
2. Next, go to the Transitions(!) pane and enter the string "alpha". See the filter correctly at work.
3. Delete the filter input character by character, notice that the filtered list doesn't update anymore.

4. Go back to the Effects(!) pane, click on "Show all effects".
5. Go to the Transitions(!) pane again, enter "alpha", and then delete this input character by character. Watch the filtered list being updated now as it should.

If this isn't an 11 at the crazyness meter for bug reports, I don't know. :D
Comment 7 Unknown 2016-01-26 21:11:45 UTC
Does it also need to be 3:00 in the morning, at 76 degrees F, while third season of Doctor who 2005 is playing on the living room TV? ;P
Comment 8 Jean-Baptiste Mardelle 2016-02-01 20:53:08 UTC
Git commit 6341d18dc03fc9ef4a8e7300fd28558a5769bea2 by Jean-Baptiste Mardelle.
Committed on 01/02/2016 at 20:52.
Pushed by mardelle into branch 'master'.

Fix transition list search widget broken

M  +1    -1    src/effectslist/effectslistview.cpp

http://commits.kde.org/kdenlive/6341d18dc03fc9ef4a8e7300fd28558a5769bea2
Comment 9 Jean-Baptiste Mardelle 2016-02-01 21:12:24 UTC
Git commit 233cad6ee3a0c83ccfd919516c4e332970329310 by Jean-Baptiste Mardelle.
Committed on 01/02/2016 at 21:11.
Pushed by mardelle into branch 'Applications/15.12'.

Fix transition list search widget broken

M  +1    -1    src/effectslist/effectslistview.cpp

http://commits.kde.org/kdenlive/233cad6ee3a0c83ccfd919516c4e332970329310
Comment 10 Wegwerf 2016-02-05 15:06:31 UTC
Jean-Baptiste, thank you very much for fixing this bug. Works now as expected, so I set this to fixed resolved.