Options/Layout/Equal split gets reverted

[egmont]

Open Options -> Layout, toggle Equal split.

Press Up or Shift+Tab to move the focus up to the Vertical/Horizontal? radiobuttons. Equal split gets reverted to its previous value.

Or: Press the Down arrow or Tab about 10 times until the focus cycles through and enters Vertical/Horizontal? radiobuttons. Again: Equal split gets reverted.

(Pressing e.g. Down and then Up properly causes the new value to stick.)

I couldn't find any other field in Options that behaves this way.

[egmont]

Related tickets: #3716, #3717.

[mooffie]

I see that radios no longer explicitly send MSG_NOTIFY to the parent when they change. It happens "by chance": in #3721 I described how pressing a key indirectly, via MSG_CURSOR, causes the widget to re-focus... which in its turn sends MSG_NOTIFY.

@Andrew:

• Why don't we go back to the happy old days when focusing a widget didn't sent MSG_NOTIFY? Or perhaps there were never such days? Whatever. Perhaps we should create MSG_NOTIFY_ON_FOCUS till the snafu is cleared up? I wasn't here when you introduced these last changes so I'm not familiar with them.

• I see that radios/buttons/chewckboxes do send_message (w->owner, w, MSG_POST_KEY, ' ', NULL) on MSG_MOUSE_CLICK. Do you happen to know what that's for?

[mooffie]

(I haven't tested the patch thoroughly.)

[mooffie]

(I attached the patch just because "a patch is worth a thousand words". I'd have preferred for the MSG_NOTIFY[MSG_FOCUS] to go away.)

[andrew_b]

Replying to mooffie:

• Why don't we go back to the happy old days when focusing a widget didn't sent MSG_NOTIFY? Or perhaps there were never such days? Whatever. Perhaps we should create MSG_NOTIFY_ON_FOCUS till the snafu is cleared up? I wasn't here when you introduced these last changes so I'm not familiar with them.

I answered in ticket:3716#comment:3.

• I see that radios/buttons/chewckboxes do send_message (w->owner, w, MSG_POST_KEY, ' ', NULL) on MSG_MOUSE_CLICK. Do you happen to know what that's for?

I can't say. It was left as is in the new mouse API.

[andrew_b]

Branch: 3718_radio_notify
changeset:e68f105e0ea790a80b8e5e2f81b2c64283630138

[andrew_b]

Merged to master: [3c8e9b389dabb59ca7a4cf06bcde685406f7c7a9].

[gnusupport]

mc --version
GNU Midnight Commander 4.8.18
Built with GLib 2.50.1
Using the S-Lang library with terminfo database
With builtin Editor
With subshell support as default
With support for background operations
With mouse support on xterm and Linux console
With support for X11 events
With internationalization support
With multiple codepages support
Virtual File Systems: cpiofs, tarfs, sfs, extfs, ftpfs, fish
Data types: char: 8; int: 32; long: 64; void *: 64; size_t: 64; off_t: 64;

If I mark Equal split to [x] it is not making any equal split in this version, it cannot be reset back to equal.

[zaytsev-work]

Of course it doesn't, the bug is fixed in master and the fix will be released with 4.8.19.