Removing the last item in a MythUIButtonList doesn't update the display

[paulh <mythtv@…>]

If you remove items from a buttonlist by calling MythUIButtonList::RemoveItem?() everything updates properly up to removing the last item which gets removed but the screen doesn't update to reflect that.

Easiest way to reproduce is in MythMusics? playlist editor screen add a few tracks to the playlist and then remove them one by one.

I can work around this for now by calling MythUIButtonList::Reset() instead of RemoveItem?() if there is only one item left in the buttonlist but it would be nice to get this fixed. I'm 95% sure this is a regression.

[Kenni Lund [kenni a kelu dot dk]]

Related to #8441?

[stuartm]

Paul I cannot reproduce in current master, I reverted the Reset() line in musiccommon.cpp so that it would only call RemoveItem?(). When I removed the last item it disappeared from screen.

Could you check whether this is still a problem for you?

[paulh]

Still behaves the same here with the Reset() removed the last item remains on screen after being removed from the list.

[stuartm]

Which UI painter are you using?

[paulh]

Replying to stuartm:

Which UI painter are you using?

OpenGL painter with MythCenter?-wide theme.

[paulh]

My guess is you can't see it because your theme is using a different <arrange> method for the buttons. The fixed arrange method works all the others are broken.

The difference seems to be fixed calls CalculateButtonPositions?() which always makes sure any unused buttons are made invisible. The other arrange methods call DistributeButtons?() which doesn't always made unused buttons invisible.

[paulh]

Proposed patch - if there are no items make all the buttons invisible

[Paul Harrison <pharrison@…>]

In d982be7d414da2678521137e08849186921a8369/mythtv:

MythUIButtonlist: fix removing the last item using RemoveItem?()

All arrange methods except the fixed one failed to make the last button
invisible when the last item was removed.

Fixes #11285.

[Paul Harrison <pharrison@…>]

In fcfe312b33653772346012973282b5750f38d406/mythtv:

MythUIButtonlist: fix removing the last item using RemoveItem?()

All arrange methods except the fixed one failed to make the last button
invisible when the last item was removed.

Fixes #11285.

(cherry picked from commit d982be7d414da2678521137e08849186921a8369)

[Paul Harrison <pharrison@…>]

In 2f3cd3cfc36a44c6676fcdb6e48437ef627b237c/mythtv:

MusicCommon?: remove the workaround to remove the last track

Refs #11285.

[Paul Harrison <pharrison@…>]

In 5d4fb28fc096f950df3a6a56b81dc9456dcdf32c/mythtv:

MusicCommon?: remove the workaround to remove the last track

Refs #11285.

(cherry picked from commit 2f3cd3cfc36a44c6676fcdb6e48437ef627b237c)