id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	blockedby	blocking	branch_state	votes
3721	Cleanup radio widget's MSG_CURSOR	mooffie	andrew_b	"While investigating #3719 I discovered a ""bug"" in the radio widget:

Whenever a key is pressed (MSG_KEY), the widget draws itself to reflect the new state. But instead of doing this drawing explicitly, it does this indirectly by focusing the widget in its MSG_CURSOR. This is a vestige from the old days.

I'm attaching a bug to fix this.

Why is this a problem? Because:

1. MSG_CURSOR has one defined purpose. It shouldn't be piggybacked for other things. This causes trouble:

2. One way to fix #3719 is simply by adding ""case MSG_FOCUS: r->pos = r->sel"". This resets the active item to the selected one whenever the user moves the focus to the widget. But since MSG_FOCUS is outrageously sent in MSG_CURSOR also, this reset will be done after each key press, which means that the active item is stuck...
"	defect	closed	minor	4.8.19	mc-core	master	fixed					merged	committed-master