|Summary:||[Regression] Dolphin 2.0 details view background lags on resize (video attached).|
|Product:||[Applications] dolphin||Reporter:||Nikita Skovoroda <chalkerx>|
|Component:||view-engine: details mode||Assignee:||Peter Penz <peter.penz19>|
|Latest Commit:||Version Fixed In:||4.8.0|
The video of the bug.
Description Nikita Skovoroda 2012-01-08 10:45:28 UTC
Created attachment 67576 [details] The video of the bug. Version: 2.0 (using Devel) OS: Linux The alternate background (on even rows) lags behind when resizing the window in horizontal direction. This did not happen in old dolphin, this does not happen in kopete. Reproducible: Always Steps to Reproduce: Enable details view. Resize the dolphin window in horizontal direction (fast). Observe the even rows background. Actual Results: Contents are resized instantly (as fast as the window), but background lags behind. Expected Results: Background should extend instantly. See the video. Video has special colorscheme applied to make it easier to spot the lag, but the lag happens on every colorscheme and can be easily spotted on the default one.
Comment 1 Nikita Skovoroda 2012-01-08 19:33:50 UTC
Can this be fixed before 4.8.0 release (to not introduce a regression)? It looks like a trivial bug.
Comment 2 Peter Penz 2012-01-08 21:29:32 UTC
Currently there are still quite a lot if other issues that need to be fixed first and have a higher priority. I also expect that until yet unnoticed issues will pop up after the 2.0 release - if only issues like this are left in Dolphin, I'll grep a beer and will be glad to fix it ;-) Seriously: I guess I hope I can fix this during the 4.8.x releases.
Comment 3 Nikita Skovoroda 2012-01-09 04:15:30 UTC
Ok, thank you ).
Comment 4 Nikita Skovoroda 2012-01-17 02:15:59 UTC
Created attachment 67914 [details] Trivial fix. This is (as it seems to me) a proper soultion to this problem This patch extends the background area of kitemlistwidget alternate row background to the width of the parent widget, not the current kitemlistwidget width, so the background is painted propertly. This patch works perfectly for me.
Comment 5 Nikita Skovoroda 2012-01-17 02:18:21 UTC
I hope this patch would be merged for 4.8.0, it is a rather annoying graphical glitch.
Comment 6 Peter Penz 2012-01-17 22:44:58 UTC
Git commit 468ea6c70a38511d2ed1d0f1c26ca30901e27c52 by Peter Penz. Committed on 17/01/2012 at 21:08. Pushed by ppenz into branch 'master'. Do a synchronous layout for dynamic item sizes when changing the geometry This prevents a delayed update when resizing the window e.g. in the details view. As no animated item position changes can happen for dynamic item sizes the synchronous relayout is fine from a performance point of view. FIXED-IN: 4.8.0 M +46 -39 dolphin/src/kitemviews/kitemlistview.cpp M +0 -3 dolphin/src/kitemviews/kitemlistview.h http://commits.kde.org/kde-baseapps/468ea6c70a38511d2ed1d0f1c26ca30901e27c52
Comment 7 Peter Penz 2012-01-17 22:45:53 UTC
Git commit ba8908e7c019bff1d92c6ea7465cbbd18cc63883 by Peter Penz. Committed on 17/01/2012 at 21:08. Pushed by ppenz into branch 'KDE/4.8'. Do a synchronous layout for dynamic item sizes when changing the geometry This prevents a delayed update when resizing the window e.g. in the details view. As no animated item position changes can happen for dynamic item sizes the synchronous relayout is fine from a performance point of view. FIXED-IN: 4.8.0 M +46 -39 dolphin/src/kitemviews/kitemlistview.cpp M +0 -3 dolphin/src/kitemviews/kitemlistview.h http://commits.kde.org/kde-baseapps/ba8908e7c019bff1d92c6ea7465cbbd18cc63883