Library repaint / refresh is slow with Preview coulmn enabled

Bug #1796765 reported by ronso0
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
High
Unassigned

Bug Description

1
* enable Preview column in the Tracks pane
* clear search bar to show all tracks (let's say about 100 or more;)
* toggle the FX section on/off
 (or any other skin section above stacked on top of the library)
> library and the toggled region repaint jerkily

2
* hide Preview column
> skin toggling works fluidly

3
* enable Preview column
* clear the Tracks table by searching some random string
(or go to some empty playlist)
> skin toggling works fluidly

Skin toggling works equally fast in tests 2 and 3, whil it's noticeably slower with visble Preview buttons.
It's also quick if I fix the height of the library so that it wouldn't resize when any skin region above is toggled.
- it doesn't matter if PNGs or SVGs are used for the Preview button.
- it doesn't matter if the library is loaded as Singleton or regular template

Basically that's the only GUI performance regression I notice after switching to qt5. The rest appears to flow at least as nice as with qt4, and shutdown (skin desctruction) is way faster now.

Tags: gui library
Revision history for this message
Faldrian (faldrian) wrote :

I encountered the same problem and after disabling the preview column the gui works fine (thank you!).
I'm using a current KDE Plasma desktop (nothing exotic), so I think more users will be impacted by this.

Not only does the region repaint like crazy, this will block the UI for up to a second if you have much screen estate and many tracks are visible. It seems that the repaint events run from bottom to top and repeat in a loop. Each loop has one or two elements (from the bottom) less to repaint until all are repainted. Maybe this helps?

Changed in mixxx:
status: New → Confirmed
importance: Undecided → High
milestone: none → 2.2.1
Revision history for this message
Daniel Schürmann (daschuer) wrote :

Maybe this is already fixed in the 2.2 branch and master.

On which distro are you? On Ubuntu, you can temporary switch to master using the nightly PPA to confirm that:
https://launchpad.net/~mixxx/+archive/ubuntu/nightlies

The other bug is:
https://bugs.launchpad.net/bugs/1812763

Revision history for this message
Daniel Schürmann (daschuer) wrote :

Is this fixed now with the 2.2.1 release?

Changed in mixxx:
status: Confirmed → Incomplete
Revision history for this message
ronso0 (ronso0) wrote :

works great with 2.2.2!

Changed in mixxx:
status: Incomplete → Fix Committed
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/9467

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.