Selected items aren't shown in MUI 5.0-2017R4

[jPV]

Summary

I got a report that my Hollywood program doesn't show the selected items in a listview. Items don't get highlighted if I select them programmatically, although they are in the selected state if I check them by the GetSelection method.

My OS4.1FE default MUI version does show them as expected, but the problem seems to be with newer MUI versions, 5.0-2017R4 in this case.

Steps to reproduce

1. Try this example program: ​http://jpv.wmhost.com/testi/select/selectiontest (and have this in the same dir: ​http://jpv.wmhost.com/testi/select/muiroyale.hwp)

Expected results

You see two selected items in the list

Actual results

None of the items seem to get selected visibly

[Thore Böckelmann]

I run the demo like this

hollywood selectiontest

But all I get is an error requester telling me about a general syntax error in line 1.

But to demonstrate that selecting items from code is definitely working I modified the DragnDrop demo a bit. Find the AmigaOS3 binary plus the source attached here.

[Thore Böckelmann]

modified DragnDrop demo for AmigaOS3

[Thore Böckelmann]

source code of modified DragnDrop demo

[jPV]

Replying to Thore Böckelmann:

I run the demo like this

hollywood selectiontest

But all I get is an error requester telling me about a general syntax error in line 1.

"selectiontest" is a compiled executable (OS4 binary), not an applet. So don't run it with Hollywood, just launch it like any other program.

[Thore Böckelmann]

Hm, strange. The modified DragnDrop demo definitely selects the desired entries visibly, while your demo does not. Although both demos produce the same debug output during the artificial selection for me.

I'll have to investigate a bit further.

[Thore Böckelmann]

Ok. Is it possible you are trying to do multiselection on a list which does not have multiselection enabled? That is the only explanation.

MUI's list item background selection works like this:
multiselection enable & selected item → selected background
active item → active background
otherwise → normal background

Since you do a selection on a non-multiselection list the first two tests will always fails and only the last resort "normal background" will be applied.

I must admit that List class is very inconsistent here. A non-multiselection list must not have more than one single selected entry, but the current implementation allows this, although it is not documented.

I will change List class' behaviour accordingly to act consistently for non-multiselection lists.

[Thore Böckelmann]

In 6304:

• List.c: calling MUIM_List_Select on a non-multiselection list will keep only the last selected entry in selected state to be consistent with the MUIA_List_MultiSelect attribute. This closes #386.