| Summary: | Maintain setting for "show only matches" option | ||
|---|---|---|---|
| Product: | [Applications] amarok | Reporter: | hvm <hvm2hvm> |
| Component: | Playlist | Assignee: | Amarok Developers <amarok-bugs-dist> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | teo |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | 2.9 | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/amarok/43f66b35d4e389044970565deacef13fa6470842 | Version Fixed In: | |
| Sentry Crash Report: | |||
|
Description
hvm
2013-08-30 21:34:34 UTC
OK, so the option does get saved but not everytime. AFAICT when I quit amarok manually it gets saved, when I just shutdown the PC it doesn't get saved. So... maybe it's a distribution problem (i.e. depends on the way shutdown is handled) - I'm using Kubuntu 13.04 And which exact Amarok version do you use? This might well be solved in 2.8 already It's Amarok 2.7 but like I said, I just started amarok and the setting was saved. I don't know exactly why it dind't earlier but if it happens again I will come back with details. You should upgrade, there have been improvements on the session saving in 2.8 Can you point me to the relevant commits or discussion please? I haven't heard of any such changes. Git commit 744b516c9c47643ab6fb2592371c0016b3fae629 by Mark Kretschmann. Committed on 02/09/2013 at 09:45. Pushed by markey into branch 'master'. Sync playlist search option right after setting. M +3 -1 src/playlist/ProgressiveSearchWidget.cpp http://commits.kde.org/amarok/744b516c9c47643ab6fb2592371c0016b3fae629 Git commit cbae195a5e784833644cf73290f93ad44cc3adcd by Mark Kretschmann. Committed on 02/09/2013 at 09:48. Pushed by markey into branch 'master'. Revert "Sync playlist search option right after setting." This reverts commit 744b516c9c47643ab6fb2592371c0016b3fae629. The patch was complete nonsense, sorry. M +1 -3 src/playlist/ProgressiveSearchWidget.cpp http://commits.kde.org/amarok/cbae195a5e784833644cf73290f93ad44cc3adcd Git commit e44eedd6c42526d4e4cdc14d1bdf18e6660a46ac by Mark Kretschmann. Committed on 02/09/2013 at 10:55. Pushed by markey into branch 'master'. Sync playlist search config instantly. M +5 -6 src/playlist/ProgressiveSearchWidget.cpp M +0 -5 src/playlist/ProgressiveSearchWidget.h http://commits.kde.org/amarok/e44eedd6c42526d4e4cdc14d1bdf18e6660a46ac Git commit fcf3dbb95158951045b35f4534cfccf3a0e91506 by Vedant Agarwala, on behalf of Mark Kretschmann. Committed on 02/09/2013 at 09:45. Pushed by vedanta into branch 'tagguessing'. Sync playlist search option right after setting. M +3 -1 src/playlist/ProgressiveSearchWidget.cpp http://commits.kde.org/amarok/fcf3dbb95158951045b35f4534cfccf3a0e91506 Git commit 43f66b35d4e389044970565deacef13fa6470842 by Vedant Agarwala, on behalf of Mark Kretschmann. Committed on 02/09/2013 at 10:55. Pushed by vedanta into branch 'tagguessing'. Sync playlist search config instantly. M +5 -6 src/playlist/ProgressiveSearchWidget.cpp M +0 -5 src/playlist/ProgressiveSearchWidget.h http://commits.kde.org/amarok/43f66b35d4e389044970565deacef13fa6470842 Git commit c8df38f04f9d73ab9ebe4a3a51d8e29edf1d467a by Vedant Agarwala, on behalf of Mark Kretschmann. Committed on 02/09/2013 at 09:48. Pushed by vedanta into branch 'tagguessing'. Revert "Sync playlist search option right after setting." This reverts commit 744b516c9c47643ab6fb2592371c0016b3fae629. The patch was complete nonsense, sorry. M +1 -3 src/playlist/ProgressiveSearchWidget.cpp http://commits.kde.org/amarok/c8df38f04f9d73ab9ebe4a3a51d8e29edf1d467a |