history920.txt wrote:12.03.18 Fixed: Button bar: When shown as dropdown menu, items without an icon are not displayed as pressed (e.g. sort by name button when list sorted by name) (32/64)
history920.txt wrote:13.04.18 Fixed: Button bar, paste button from clipboard: Remove all trailing spaces, they are often added when copying button bar data from a Web browser (32/64)
Both
no icon button issue and
button code with trailing spaces issue I can confirm fixed, tested in TC 9.20 beta 1 on Vista 32-bit.
There is however one issue left with one example not mentioned in above posts. If TC is without TOTALCMD.INC, it's possible to create a button for internal command like cm_SwitchLongNames, but since there is no INC file to tell TC to what command ID it should match, TC will create it with -1 ID (-1 indicates that the command is an external command, and TC without TOTALCMD.inc will try to launch cm_ command as an external app).
If copied from such TC instance, the button code will be
Code: Select all
TOTALCMD#BAR#DATA
cm_SwitchLongNames
*
Turn &long names on and off
-1
If pasted in Total Commander instance which has access to TOTALCMD.INC, TC will prefer internal command over ID, so command 2010 will be actually executed.
So the issue is: if such button is pasted into TC instance which has access to TOTALCMD.INC, that Total Commander shows the pressed state for such button with cm_ command and -1 ID only after restart. I asked by email to implement a check for pressed state while button is pasted, so it will immediatly show the state of action button actually makes. So far Christian Ghisler's point is that such button is invalid, so no need to implement such check. But I think since such button works, TC should show its state too.